Online Accounts: Account plugins

Code review comment for lp:~jpds/account-plugins/linkedin-plugin

  1. linkedin-plugin
  2. Merge into trunk
Revision history for this message
Alberto Mardegan (mardy) wrote :

Looks good, thanks! I'll experiment some more with signon-ui, to see if we can fix the issue with the missing scrollbars.
Meanwhile, could you please keep the files in alphabetical order:

8 - data/providers/windows-live.provider.in.in
9 + data/providers/windows-live.provider.in.in \
10 + data/providers/linkedin.provider.in.in

(there are more places where this should be changed, please see the diff below).

review: Needs Fixing

« Back to merge proposal