-
Notifications
You must be signed in to change notification settings - Fork 24
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
feedback when importing elm project fails #11
Comments
I think putting there a question mark in a circle, which on mouseover shows the following text would be an ok solution:
So that we don't have to complicate the code for an edge case. |
it'd be good to display that text! i think we still need to report errors in a few places, including this one. you may get rate limited, in which case we should encourage getting a github token, but maybe your wifi suddenly fails, in which case you just need to try again once you're online. there's also no feedback if you try to import a DOT file and the file fails to parse. this has confused me a few times. maybe we can have "toast" notifications popup, and use those notifications for a variety of errors. |
i was using another app, and this popped up when i hit the GitHub rate limit. just wanted to record this for inspiration |
i'm unable to import large elm project, at least without providing a github token. for example, importing the dependency graph for
stil4m/elm-syntax
causes a rate limit error before all the files have been fetched.currently, the only way to know what happened is by looking in the dev tools. it'd be nice if we reported/displayed errors in this situation.
EDIT: just to confirm, providing a github token gets past the rate limit. the error message should point you towards getting a token of your own.
The text was updated successfully, but these errors were encountered: