Closed davidmehren closed 3 years ago
I tested it with exact this browser version aaaand.. can't reproduce. Something must be wrong with your setup.
For a short time we had a broken service worker. Check if it is still registered in your browser (dev tools > Applications > Service Workers > unregister every service worker there)
This likely happens because we changed editorConfig
local-storage syntax from editorMode: 1, 2, 3
to editorMode: view, both, edit
. Clear your local-storage and try it again.
@ErikMichelson maybe we should change the fallback. And write it back if we use it.
Well we currently do not verify the data at all (return JSON.parse(stored) as EditorConfig
). I doubt that we really need to test each property - especially as this problem only appears for regular ui-test users.
Also note that once you select one of the editor modes, the locally stored config will be corrected.
Yes, but a type cast doesn't verify the data we read. I think it would be better to check the data for plausability.
For API response data validation we're tracking the progress in hedgedoc/hedgedoc#2910. As this seems to be solved, I'm going to close this now.
Description After pasting
The dev tools show a variety of messages:
This behavior persisted over multiple reloads with cache disabled.
https://ui-test.hedgedoc.org/n/features
into the Chrome Omnibox, the frontend only shows a big black box:Manually selecting the "both" view mode switched to the correct mode after a short delay and caused two more warnings:![grafik](https://user-images.githubusercontent.com/5023650/111382358-2bf3f200-86a7-11eb-9288-610b18434988.png)
After that the issue was not reproducible anymore.
To Reproduce
Expected behavior The frontend shows one of the three view modes without the user having to manually select one.
Your Setup: