-
Notifications
You must be signed in to change notification settings - Fork 50
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
Fix invalid module import #1054
Conversation
Actually, I am confused now. If this is in webcode, and if this is for displaying the bottom panel in region overview, then wouldn't it have broken earlier and everywhere if it were truly broken? Do we need a sandbox to test this code change, or is it already obvious that it couldn't have worked anyway? |
In the sandbox I can see this part of code being executed (when editing region overview menu entry and debug messages), but I still cannot see the difference with/without the |
All right. Let's |
For history. The reason we decided to make this change was because of this error in the logs:
|
Description
Remove invalid module import syntax that is generating server errors.
See this slack thread for context.
Sandbox
http://wp-np2-1d.ebi.ac.uk:1640