← Back to team overview

launchpad-reviewers team mailing list archive

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