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

Add a refresh interval to the origin exports #1792

Merged
merged 6 commits into from
Dec 16, 2024

Conversation

CannonLock
Copy link
Contributor

@CannonLock CannonLock commented Dec 3, 2024

  • UI and UI backing updates to make sure its up to date

The current issue is that when you redirect from a corrected registration in the registry back to the origin you were registering the initial query from the origin is so fast that it misses the changes.

This adds a UI loading bar that waits 5 seconds for a requery. If for some reason it takes longer then 5 seconds I also auto refresh these metrics every 5 seconds in the background so things will just switch over automatically in the UI.

- UI and UI backing updates to make sure its up to date
@CannonLock
Copy link
Contributor Author

CannonLock commented Dec 3, 2024

Screen.Recording.2024-12-03.at.3.52.09.PM.mov

@CannonLock
Copy link
Contributor Author

Closes #1520

@CannonLock CannonLock requested a review from h2zh December 4, 2024 14:47
@h2zh
Copy link
Collaborator

h2zh commented Dec 4, 2024

This loading bar pops up every time when the origin page reloads. Is that an expected behavior?

@CannonLock CannonLock linked an issue Dec 4, 2024 that may be closed by this pull request
Only indicate loading if returning from registry
Tweak language and header updates
- Toss component in Suspense boundary
@CannonLock
Copy link
Contributor Author

CannonLock commented Dec 10, 2024

@h2zh I didn't think to remove it all the time. I pushed a couple commits to address this. Now it knows when it is being returned to from the registry and will only poll/show the loading bar then.

Copy link
Collaborator

@h2zh h2zh left a comment

Choose a reason for hiding this comment

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

LGTG

@h2zh h2zh added the origin Issue relating to the origin component label Dec 12, 2024
@CannonLock CannonLock merged commit 11232c8 into PelicanPlatform:main Dec 16, 2024
22 of 23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
origin Issue relating to the origin component
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Namespace Registration Completion Polling
2 participants