Closed Bug 967008 Opened 10 years ago Closed 10 years ago
Accounts should return the account info right away even if the account isn't verified
Right now, if the account in the device is not verified, we check with the server the account status before returning the account information. We should return the account information right away and check the verification status in the background.
Jed this seems to work for me. Could it be this simple?
Attachment #8395897 - Flags: feedback?(jparsons)
Comment on attachment 8395897 [details] [diff] [review] 967008-getAccounts-unverified.patch This certainly seems like it will do what comment 0 is asking. So f+ on the direction. If there are no longer any functions waiting on promises from verificationStatus(), maybe it shouldn't return anything. In which case, verificationStatus() will want some reworking to make it simpler. In the case of errors in verificationStatus(), such as 'offline', would they now just get logged? (Again, no promises to reject with _error().) Also, don't forget to update the tests.
Attachment #8395897 - Flags: feedback?(jparsons) → feedback+
Per feedback I converted verificationStatus() to have no return value; errors are now logged. I updated three of the affected tests; the fourth did not seem worth saving.
Comment on attachment 8396741 [details] [diff] [review] 967008-getAccounts-return.patch Review of attachment 8396741 [details] [diff] [review]: ----------------------------------------------------------------- Thanks, Sam. Looks good to me. I wonder whether there couldn't be a better name than 'verificationStatus' for this function now. Maybe 'checkVerificationStatus' or 'updateVerificationStatus' or something. If you can think of a great name, go ahead and change it. But it's ok by me if it goes in like this. r=me
Attachment #8396741 - Flags: review?(jparsons) → review+
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → 1.5 S1 (9may)
You need to log in before you can comment on or make changes to this bug.