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(PEER-752): reproduce type error #28

Merged
merged 1 commit into from
Aug 29, 2024

Conversation

amaralc
Copy link
Owner

@amaralc amaralc commented Aug 29, 2024

What and why was modified?

  • Dynamic imported module added using React.lazy to try to reproduce and fix the issue TypeError: Failed to fetch dynamically imported module vitejs/vite#11804 for the following use case:
    • WHEN user loads initial page before a new deployment is made;
    • AND a new deployment removes a dynamically imported module that was referenced in the previous deployment;
    • AND user clicks a link that is available in his stale version;
    • THEN user experiences an error TypeError: Failed to fetch dynamically imported module;
  • Chunk file name and path outputs were customized using rollulpOptions to avoid the file update issue;
  • A fallback component was added to the router error component;

@amaralc amaralc merged commit b879262 into trunk Aug 29, 2024
3 of 4 checks passed
Copy link

sonarcloud bot commented Aug 29, 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.

1 participant