Thanks to visit codestin.com
Credit goes to github.com

Skip to content

use doctrine.middleware tag for registering middlewares#300

Merged
dmaicher merged 1 commit into
masterfrom
middleware_priority
May 28, 2024
Merged

use doctrine.middleware tag for registering middlewares#300
dmaicher merged 1 commit into
masterfrom
middleware_priority

Conversation

@dmaicher
Copy link
Copy Markdown
Owner

@dmaicher dmaicher commented May 28, 2024

Fixes #251

@dmaicher dmaicher force-pushed the middleware_priority branch from 502600e to e0199eb Compare May 28, 2024 15:25
@dmaicher dmaicher force-pushed the middleware_priority branch from e0199eb to eca12bc Compare May 28, 2024 15:26
@dmaicher
Copy link
Copy Markdown
Owner Author

dmaicher commented May 28, 2024

@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
Copy Markdown

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
@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