When you try to log in to an instance you don't have permission to log in to, we should give you an explicit message and direct you to the list of instances which you have accounts on.
Description
Description
Revisions and Commits
Revisions and Commits
| Restricted Differential Revision | Restricted Diffusion Commit | ||
| Restricted Differential Revision | Restricted Diffusion Commit | ||
| Restricted Differential Revision | Restricted Diffusion Commit | ||
| rP Phabricator | |||
| D14050 | rP9d0332c2c0a4 Modernize OAuthserver and provide more context on "no permission" exception | ||
| D14049 | rP1fc60a9a6e70 Modularize Aphront exception handling | ||
| D14047 | rP20ce1a905f3a Replace AphrontUsageException with AphrontMalformedRequestException | ||
| Status | Assigned | Task | ||
|---|---|---|---|---|
| Open | None | T9303 Improve Phacility Onboarding/NUX | ||
| Resolved | epriestley | T7173 Improve "you can not log in to this instance" workflow | ||
| Resolved | None | T7169 Visibility of OAuth apps is not set correctly or is not checked correctly |
Event Timeline
Comment Actions
After T7169, this gives you a reasonable error message, but it could be a little nicer.
Comment Actions
After T9297 / D14017 , it would be nice to do this:
- Check if you could log in to the instance if you verified one of your unverified email addresses.
- Tell you that you should go do that.
Specifically, the path-of-least-resistance workflow will be:
- Register.
- Try to log in to your instance.
- Sort of confusing error?
- Maybe go check your email later?
- Click verification link.
- Things work OK now.
This isn't great. It would be better if that error said "You can log in as soon as you verify your email address.".
