-
Notifications
You must be signed in to change notification settings - Fork 14
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
Site is refreshing after resizing or dragging event #207
Comments
@that-one-arab can you pick this up? |
Yup leave it to me. @tyler-dane although I didn't see a reproduction video and the steps leave room for misinterpretation, can you please record a reproduction video or add more details to reproduce steps? |
@that-one-arab I don't have a video handy, but added a few more details |
Hi, |
Hey @omarNaifer12 thank you for your efforts! Usually it charges your credit card and refunds it immediately, I believe it is to prevent misuse on Google's end of things. |
Moved this from In Progress to the backlog, as @that-one-arab mentioned how he was unable to reproduce the bug. I'll keep an eye out for this and try to capture more details if it keeps happening on my end |
Reproduced, added video, and moved this back to Ready |
Where did this happen?
Production app, Local code with an up-to-date main branch
Expected Behavior
Resizing or dragging an event shouldn't trigger a page refresh
Current Behavior
Full page refresh happens sporadically after resizing or dragging an existing event
output2.mov
UI
The UI automatically refreshes as part of the error handling for our custom 601 error, so it's hard to show how this looks beyond the above video.
You can disable that refreshing if you'd like to see the state of the UI before the refresh happens
Backend
This is what you'll see in the backend logs
DB
If you look in the mongo DB, you'll see the event is missing a
user
propertySteps to Reproduce
Possible Solution
This is caused by the request event payload missing the
user
value. Thisuser
is used by the backend inevent.queries:updateEvent()
to identify the event to update.Because the
user
value is missing, the event isn't found, which triggers the backend to throw the following errorThe frontend obliges, by refreshing the entire page in an attempt to resync the redux store. (In fact, this isn't necessary, as the Redux store likely wasn't the problem)
Context
This is a very intrusive interruption that waste's a user's time and makes them have less confidence in the app.
The text was updated successfully, but these errors were encountered: