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

Fix service definition #39

Closed
wants to merge 2 commits into from
Closed

Fix service definition #39

wants to merge 2 commits into from

Conversation

relthyg
Copy link
Member

@relthyg relthyg commented Mar 26, 2024

No description provided.

@relthyg relthyg changed the base branch from master to 3.x March 26, 2024 12:03
@relthyg relthyg changed the title service definition Fix service definition Mar 26, 2024
@mpdude
Copy link
Member

mpdude commented Mar 26, 2024

Wird der Service nicht schon auto-wired und ist ein Event Subscriber?

Mich überrascht, dass diese Änderungen notwendig sind bzw. dass wir bisher auf dem 3.x Branch nicht gemerkt hätten, dass sie fehlen.

@relthyg
Copy link
Member Author

relthyg commented Mar 27, 2024

Ich verstehe das auch nicht, aber ohne den PR hier wird im Symfony7-Branch von GBA-Druckerzeignisse der PolyglotListener nicht gerufen.

@mpdude
Copy link
Member

mpdude commented Mar 27, 2024

Dann lass uns da ein paar 👀👀 hinwerfen.

@mpdude mpdude deleted the branch 3.x March 27, 2024 17:36
@mpdude mpdude closed this Mar 27, 2024
@mpdude mpdude deleted the service-definition branch April 4, 2024 16:51
@mpdude
Copy link
Member

mpdude commented Apr 4, 2024

@relthyg Kannst du bestätigen, dass das Problem sich in Luft aufgelöst hat?

@relthyg
Copy link
Member Author

relthyg commented Apr 5, 2024

@mpdude Nein, ich brauche weiterhin die vier Tags als doctrine.event_listener, und wenn ich die Doku an der Stelle richtig verstehe, liegt das hier dran.

@relthyg
Copy link
Member Author

relthyg commented Apr 5, 2024

Geht weiter in #51

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

Successfully merging this pull request may close these issues.

2 participants