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

Remove synchronous route loaders #91

Closed
pshenmic opened this issue Feb 25, 2024 · 1 comment
Closed

Remove synchronous route loaders #91

pshenmic opened this issue Feb 25, 2024 · 1 comment
Assignees
Labels
frontend Related to fronted (UI)

Comments

@pshenmic
Copy link
Owner

Issue

Current page routes data implemented via route loaders, required data to be fetched first until page is rendered. That makes page synchronous, and also breaks the application in case there is an error during API request. We should remove all route loaders and make page all requests on the page asyncronous, showing the user correct UI elements (pending, success, error messages)

Possible solution

  • Remove all route loaders
  • Implement asyncronous data loading (same as we send state transition on decoding in transaction route)
@pshenmic
Copy link
Owner Author

pshenmic commented Jun 2, 2024

#133
#135

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
frontend Related to fronted (UI)
Projects
Status: Done
Development

No branches or pull requests

2 participants