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

[4.x] Fixes for asset/term reference updater strictness #9878

Merged
merged 4 commits into from
Apr 15, 2024

Conversation

jesseleite
Copy link
Member

@jesseleite jesseleite commented Apr 12, 2024

This PR takes over from where #7555 and #9790 left off, but addresses more edge cases.

  • Asset reference updater
    • Ensure we handle string/array values regardless of max_files configuration
    • Ensure we handle string/array values regardless of save_html configuration
    • Add test coverage for the above
  • Term reference updater
    • Ensure we handle string/array values regardless of max_items configuration
    • Ensure we handle string/array values for scoped terms regardless of max_items configuration
    • Add test coverage for the above

Closes and fixes #7555
Closes #9790
Fixes #9701
Fixes #7547
References #7555
References #8114

@jesseleite jesseleite marked this pull request as ready for review April 12, 2024 22:11
@jesseleite jesseleite requested a review from jasonvarga April 12, 2024 22:11
@jasonvarga jasonvarga merged commit b929d3b into 4.x Apr 15, 2024
36 checks passed
@jasonvarga jasonvarga deleted the fix/reference-updater-strictness branch April 15, 2024 13:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants