Skip to content

fix: symfony uuid compatibility #28

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.

Sign up for GitHub

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

Jump to bottom
Closed
manuel-gamma wants to merge 1 commit into from
Closed

fix: symfony uuid compatibility #28

manuel-gamma wants to merge 1 commit into from

Conversation

@manuel-gamma
Copy link

@manuel-gamma manuel-gamma commented Jun 16, 2025

  1. add symfony/uid to composer suggest
  2. use match_phrase for found item with uuid
  3. update functional tests

#27

@manuel-gamma
fix: symfony uuid compatibility
b2cda0b 
add symfony/uid to composer suggest
use match_phrase for found item with uuid
update functional tests

Locastic#27
@manuel-gamma manuel-gamma force-pushed the bugfix/symfony-uuid branch from abe6b60 to b2cda0b Compare June 23, 2025 14:11
@paullla
Copy link
Member

paullla commented Jun 24, 2025

hey @manuel-gamma, this can be easily done by decorating DataProvider/CurrentDataTrackerProvider. I don't think we should add a new dependency to the library which is not directly needed for the library logic. Maybe instead add this part to the docs and contribute in that way? :)

@manuel-gamma
Copy link
Author

manuel-gamma commented Jun 24, 2025

Thanks, I will suggest a documentation update in another PR

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

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@manuel-gamma @paullla