Code review comment for lp:~rpadovani/reminders-app/1392197

Revision history for this message
Michael Zanetti (mzanetti) wrote :

@David, no this doesn't break logging in with sandbox accounts, however, thinking about it, this would list enabled sandbox accounts AND production accounts in the list. Given that we only switch between those two using a cmdline argument, I guess that's still bogus. Changing my vote to needs fixing.

Also, as David said, please try to fix the "tap to authorize"

review: Needs Fixing

« Back to merge proposal