Skip to content
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

UI hang on missing config.py #720

Closed
spyysalo opened this issue Feb 17, 2012 · 4 comments
Closed

UI hang on missing config.py #720

spyysalo opened this issue Feb 17, 2012 · 4 comments

Comments

@spyysalo
Copy link
Member

If config.py is not present, appropriate error messages ("Missing file config.py ...") are shown, but the UI hangs. May be due to failure to deal with client-server exception configurationError.

@spyysalo
Copy link
Member Author

Seems other client-server exceptions can result in this state also.

@amadanmath
Copy link
Contributor

UI should hang if the configuration file is not present. The only thing that the client should do in this situation is inform the user they should go yell at the admin. What else would you expect? The only complaint I have in this scenario is that the client does not die soon enough, spamming the message three times, and that the server returns the exception without copying the action value in the response which results in further spam.

@amadanmath
Copy link
Contributor

Ah, I did not pull. Still, should die sooner. On it.

@spyysalo
Copy link
Member Author

@amadanmath : not doing anything is fine, but blocking the user from either reloading the page from the KB or from seeing the messages from the server (i.e. disabling mouseover on the "info") is not.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants