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.

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

use doctrine.middleware tag for registering middlewares #300

Merged
merged 1 commit into from
May 28, 2024
Merged

use doctrine.middleware tag for registering middlewares #300

merged 1 commit into from
May 28, 2024

Conversation

dmaicher
Copy link
Owner

Fixes #251

@dmaicher
Copy link
Owner Author

@MatTheCat could you maybe test this? For now I set 100 as priority for the middleware. So if you choose a value > 100 for your EnableForeignKeysmiddleware then it should work 🤞

@MatTheCat
Copy link

MatTheCat commented May 28, 2024 •
edited
Loading

@dmaicher I can confirm it makes #299’s test pass when setting EnableForeignKeys’s priority above 100. 🎉

@dmaicher dmaicher marked this pull request as ready for review May 28, 2024 15:41
@dmaicher dmaicher merged commit 1f81a28 into master May 28, 2024
11 checks passed
@dmaicher dmaicher deleted the middleware_priority branch May 28, 2024 15:41
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.

Leverage priority on doctrine.middleware tags?
2 participants
@dmaicher @MatTheCat