-
-
Notifications
You must be signed in to change notification settings - Fork 7
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
beta.pybricks.com fails to load #689
Comments
I think the bug is actually with the previous releases - not something new introduced in this release (and possibly the same bug as #686). The There is a line in the webpack config (inherited from Create React App) that specifically excludes this file: Will need to dig further. |
Looks like the problem was inadvertently introduced in pybricks/pybricks-code@9cdd102. |
This caused updates to not work if main.js was not in the browser cache since it was being excluded from the service worker. Issue: pybricks/support#689
This has been fixed in v3.2.0b3 (Pybricks Beta v2.0.0-beta.5). But I'm going to leave the issue open for a while since people who have used older versions of Pybricks Beta will still experience this bug when upgrading to the fixed version (because the old buggy version is already on their computer). |
Not sure it is the same, but maybe this was also happening on Pybricks Code. See https://bricks.stackexchange.com/questions/17234/pybricks-on-android-tablet/17322#17322 |
This particular bug has only been released on https://beta.pybricks.com, not https://code.pybricks.com like mentioned in the stackexchange question. But the same workaround is probably still applicable. |
I think it has been long enough now that we can close the issue. |
Describe the bug
With the update today, https://beta.pybricks.com shows a blank page.
This only happens if you have visited the site previously.
The console log shows the following error error:
Woraround
Refresh the page with CTRL+SHIFT+R (CMD+SHIFT+R on Mac).
Expected behavior
Pybricks Beta should not be blank and should notify that there is an updated.
Screenshots
The text was updated successfully, but these errors were encountered: