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

feat: add block participants updates #35

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

dawidlinek
Copy link
Member

Intro

The participant list in blocks did not refresh after a participant registered. After conducting an in-depth analysis, I discovered that participant data in blocks is not refreshed at all. This is a critical functionality since users need real-time updates to confirm whether someone else has already taken their place in real life.

Changes

I've added all the prefetch parameters I could change. By default, places are refreshed every second using a pooling methodology. This also happens after the window focus changes, after reconnecting, and on mounting. This ensures a smooth experience.

Future development

I also noticed that the whole event (meaning configuration I guess) is also refreshed every second. This produces quite a lot of requests and we should ask ourselves how we can optimize it.

@dawidlinek dawidlinek added the enhancement New feature or request label Dec 3, 2024
@dawidlinek dawidlinek requested a review from Rei-x December 3, 2024 17:11
@dawidlinek dawidlinek self-assigned this Dec 3, 2024
@solvro-coolify
Copy link

solvro-coolify bot commented Dec 3, 2024

The preview deployment is ready. 🟢

Open Preview | Open Build Logs

Last updated at: 2024-12-03 17:23:27 CET

@dawidlinek dawidlinek linked an issue Dec 3, 2024 that may be closed by this pull request
@Rei-x
Copy link
Member

Rei-x commented Dec 4, 2024

gpt driven opis pr'ki, grubo

Comment on lines +71 to +73
refetchOnWindowFocus: true,
refetchOnReconnect: true,
refetchOnMount: true,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ale wiesz że to nic nie zmienia bo by default tak jest?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat/odświeżenie informacji na żywo
2 participants