Skip to content
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

Open
tyler-dane opened this issue Dec 29, 2024 · 7 comments
Open

Site is refreshing after resizing or dragging event #207

tyler-dane opened this issue Dec 29, 2024 · 7 comments
Labels
bug Something isn't working

Comments

@tyler-dane
Copy link
Contributor

tyler-dane commented Dec 29, 2024

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

Notice how the page refreshes automatically after I simply click the event after dragging it.

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

Screen Shot 2024-12-31 at 6 17 48 PM

DB
If you look in the mongo DB, you'll see the event is missing a user property

Steps to Reproduce

  1. create new event
  2. drag/resize it
  3. continue until the page is refreshed
  4. view the logs and you'll see the following backend error:
app:error.handler: {"result":"Prompt Redux refresh","description":"Invalid event id (most likely)","statusCode":601,"isOperational":true}

Possible Solution

This is caused by the request event payload missing the user value. This user is used by the backend in event.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 error

 throw error(EventError.NoMatchingEvent, "Prompt Redux refresh");

The 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.

@tyler-dane tyler-dane added the bug Something isn't working label Dec 29, 2024
@tyler-dane
Copy link
Contributor Author

@that-one-arab can you pick this up?

@that-one-arab
Copy link
Contributor

that-one-arab commented Dec 31, 2024

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?

@tyler-dane
Copy link
Contributor Author

@that-one-arab I don't have a video handy, but added a few more details

@tyler-dane tyler-dane moved this from Ready to In progress in 🏗 Compass Roadmap Jan 1, 2025
@omarNaifer12
Copy link

Hi,
I’m trying to contribute, and the documentation says I need a Google Cloud account to get OAuth credentials. However, when I try to create an account, it requires a credit card, which I don’t have. Is there any other solution to contribute, or is this the only way?

@that-one-arab
Copy link
Contributor

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.

@tyler-dane tyler-dane moved this from In progress to Backlog in 🏗 Compass Roadmap Jan 2, 2025
@tyler-dane
Copy link
Contributor Author

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

@tyler-dane tyler-dane added needs more info Not ready for coding just yet and removed needs more info Not ready for coding just yet labels Jan 2, 2025
@tyler-dane tyler-dane moved this from Backlog to Ready in 🏗 Compass Roadmap Jan 2, 2025
@tyler-dane
Copy link
Contributor Author

Reproduced, added video, and moved this back to Ready

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: Ready
Development

No branches or pull requests

3 participants