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

fix(web): allow reloading the page during installation #1503

Merged
merged 4 commits into from
Jul 26, 2024

Conversation

imobachgs
Copy link
Contributor

The new useProduct hook uses the Suspense API to display a "loading" message while retrieving the
list of products. Unfortunately, that's a problem during installation: the software service won't
response with the list of products and the UI will get stuck.

This PR fixes the problem by adding a suspense option useProduct so you can decide when to use
the Suspense API.

Copy link
Contributor

@dgdavid dgdavid left a comment

Choose a reason for hiding this comment

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

Looks good to me, just two minor NP comments

web/src/queries/software.ts Outdated Show resolved Hide resolved
web/src/types/queries.ts Outdated Show resolved Hide resolved
Copy link
Contributor

@dgdavid dgdavid left a comment

Choose a reason for hiding this comment

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

You were right, it looks better now. Thanks. LGTM

@imobachgs imobachgs merged commit 7f55213 into master Jul 26, 2024
2 checks passed
@imobachgs imobachgs deleted the fix-products-suspense branch July 26, 2024 14:32
@imobachgs imobachgs mentioned this pull request Sep 20, 2024
imobachgs added a commit that referenced this pull request Sep 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants