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

Update React Query to v5 #2043

Merged
merged 4 commits into from
Feb 12, 2025
Merged

Update React Query to v5 #2043

merged 4 commits into from
Feb 12, 2025

Conversation

ChristopherChudzicki
Copy link
Contributor

@ChristopherChudzicki ChristopherChudzicki commented Feb 12, 2025

What are the relevant tickets?

Closes #1940

Description (What does it do?)

Updates react query to v5

How can this be tested?

  • There should be no functional changes. Good to check basic functionality.
  • Could also check that server-side request warnings still work, though we have tests for this. For example:
    • In frontends/main/src/app/page.tsx, comment out one of the requests inside prefetch. When you reload the page, you should see a warning in JS console.
    • In frontends/main/src/app/page.tsx, add an extra useless request inside prefetch. (E.g., learningResourceQueries.featured({ limit: 1 }). You should see a warning in JS console that a useless prefetch was made.

@@ -24,21 +24,21 @@ const learningPathKeys = {

const learningPathQueries = {
list: (params: ListRequest) =>
({
queryOptions({
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@ChristopherChudzicki ChristopherChudzicki marked this pull request as ready for review February 12, 2025 13:18
Copy link
Contributor

@jonkafton jonkafton left a comment

Choose a reason for hiding this comment

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

Huge win!

@ChristopherChudzicki ChristopherChudzicki merged commit a295f34 into main Feb 12, 2025
11 checks passed
@ChristopherChudzicki ChristopherChudzicki deleted the cc/update-react-query branch February 12, 2025 19:44
@odlbot odlbot mentioned this pull request Feb 13, 2025
4 tasks
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