-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
Saving doesn't work properly #1591
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
Comments
Welcome! 👋 Thanks for opening your first issue here! And to ensure the community is able to respond to your issue, be sure to follow the issue template if you haven't already. |
@patfr Hi, I have just done the same on my browser but the save option is working fine for me |
Thanks for reporting! I think this is a cacheing issue rather than a saving issue. The solution would be similar to how #1234 was fixed—adding a header so that the endpoint doesn't get cached ever. |
Hi, @catarak can I try to fix this issue! |
@m-e-l-u-h-a-n yes! you can go ahead and work on this issue 😄 |
@catarak I followed the dev-setup installation from develop/developer_docs/installation.md but when I start the development server using EDIT: Also tried the github login but it does not redirects to the |
If you look at the API request for login (like in the Network developer tools) when you submit your username/password, is there an error code or does it return a 200 code? |
Yes it did return a response code of 200 but, it also returned me following Upon looking at line 66 in |
Thanks for looking into this! I'm able to reproduce the same error. I believe it was introduced by #1707—will try to fix this now. |
Okay, I was actually able to figure this out! Turns out it's not an issue with #1707, it's an issue with the npm library |
Thanks, @catarak for a really quick fix 😄 ! |
@catarak I have checked for this issue and was not able to reproduce it in development mode. Although I can still reproduce it in the
|
This must have something to do with caching via Cloudflare 😕 |
Nature of issue?
Details about the bug:
The text was updated successfully, but these errors were encountered: