launchpad-reviewers team mailing list archive
-
launchpad-reviewers team
-
Mailing list archive
-
Message #30783
Re: [Merge] ~ines-almeida/launchpad:db-add-social-accounts-table into launchpad:db-devel
- Updated to the singular form, but we have a bunch of tables with PascalCase looking at recent patches?
- Added. Guruprasad had already suggested it. I didn't add it from the start because I doubt we will use it in the UI + API, but I agree it doesn't hurt to add and can be good for debugging.
- Yes, but we wont' do that right now. For now the focus is adding a new social platform in a way that is sustainable to add others later; later we might migrate those existing tables.
--
https://code.launchpad.net/~ines-almeida/launchpad/+git/launchpad/+merge/457221
Your team Launchpad code reviewers is requested to review the proposed merge of ~ines-almeida/launchpad:db-add-social-accounts-table into launchpad:db-devel.
References