Skip to content

refactor(ui): improve relationship field option loading reliability using queues #12653

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

Merged
merged 2 commits into from
Jun 2, 2025

Conversation

AlessioGr
Copy link
Member

This PR uses the new useQueue hook for relationship react-select field for loading options. This will reduce flakiness in our CI and ensure the following:

  • most recently triggered options loading request will not have its result overwritten by a previous, delayed request
  • reduce unnecessary, parallel requests - outdated requests are discarded from the queue if a newer request exist

@jacobsfletch jacobsfletch changed the title refactor: improve relationship field option loading reliability using queues refactor(ui): improve relationship field option loading reliability using queues Jun 2, 2025
paulpopus
paulpopus previously approved these changes Jun 2, 2025
jacobsfletch
jacobsfletch previously approved these changes Jun 2, 2025
@AlessioGr AlessioGr enabled auto-merge (squash) June 2, 2025 18:42
@AlessioGr AlessioGr dismissed stale reviews from jacobsfletch and paulpopus via d0d5675 June 2, 2025 19:06
@AlessioGr AlessioGr merged commit 30dd9a2 into main Jun 2, 2025
149 of 152 checks passed
@AlessioGr AlessioGr deleted the refactor/react-select-queues branch June 2, 2025 21:33
AlessioGr added a commit that referenced this pull request Jun 2, 2025
This PR introduces a few changes to improve turbopack compatibility and
ensure e2e tests pass with turbopack enabled

## Changes to improve turbopack compatibility
- Use correct sideEffects configuration to fix scss issues
- Import scss directly instead of duplicating our scss rules
- Fix some scss rules that are not supported by turbopack
- Bump Next.js and all other dependencies used to build payload

## Changes to get tests to pass

For an unknown reason, flaky tests flake a lot more often in turbopack.
This PR does the following to get them to pass:
- add more `wait`s
- fix actual flakes by ensuring previous operations are properly awaited

## Blocking turbopack bugs
- [X] vercel/next.js#76464
  - Fix PR: vercel/next.js#76545
  - Once fixed: change `"sideEffectsDisabled":` back to `"sideEffects":`
  
## Non-blocking turbopack bugs
- [ ] vercel/next.js#76956

## Related PRs

#12653
#12652
Copy link
Contributor

github-actions bot commented Jun 5, 2025

🚀 This is included in version v3.41.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants