New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Improve error reports to the user for spotify imports #423
Conversation
* Tell the user if their account is inactive i.e we aren't importing their listens. * Show errors that occured during their imports. * Show times when listens were imported last.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Apply two nit-pick fixes and merge!
|
||
{% if account.error_message %} | ||
<h3>Import Errors</h3> | ||
<p>There was an error while importing listens from your Spotify account.</p> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nitpick: use a colon here.
{% endif %} | ||
|
||
{% if not account.last_updated %} | ||
<p>We haven't gotten around to importing your listens yet, please come back later!</p> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nitpick: please check back later.
{% if not account.last_updated %} | ||
<p>We haven't gotten around to importing your listens yet, please come back later!</p> | ||
{% else %} | ||
<p>We imported your listens <abbr class="timeago" title="{{last_updated}}">{{last_updated}}</abbr> and the last song we imported was listened to <abbr class="timeago" title="{{latest_listened_at}}">{{latest_listened_at}}</abbr>.</p> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That is great!
Summary
Problem
We saved error messages for users in the db but didn't display them.
Solution
Improve error reporting and information given to user.
importing their listens.