search

sandstorm-io / sandstorm

Sandstorm is a self-hostable web productivity suite. It's implemented as a security-hardened web app package manager.
https://sandstorm.io
Other
6.72k stars 705 forks source link

"Send an e-mail invite" UX feedback is confusing #500

Open kentonv opened 9 years ago

kentonv commented 9 years ago

Things we should improve:

  1. On clicking "send", if successful, there is no feedback that the messages were sent.
  2. If the SMTP server is configured incorrectly, the feedback is "500 internal server error". It should at least suggest checking the SMTP settings.
ar-jan commented 9 years ago

I just noticed both of these while testing. In case of 2, a user is likely not to be the server administrator, it should probably also something like please contact the system administrator (which should perhaps be a configurable option, e.g. name and/or email address?).

kentonv commented 9 years ago

@ar-jan This bug is specifically about the "send email invite" UX which is part of the admin panel, so only applies to admins.

But if the "passwordless email login" box also provides poor feedback then I agree it should suggest contacting the administrator. If the user is the administrator, then hopefully they'll click the troubleshooting link...

ar-jan commented 9 years ago

The 500 internal server error can also appear for users who have received an invitation email, below the Send Login Email button. I suppose this is less likely to happen; only when the smtp server was configured correctly and an invite was sent, but the Send Login Email doesn't work because it's not an actual SMTP relay server, as in #531.

paulproteus commented 8 years ago

@neynah says this is closed, so I am going to mark it as such!

kentonv commented 8 years ago

The second part of this issue is still open. Note that this is the invite tab in the admin panel -- this is not related to the "share access" UI.