Skip to content

fix: Stop using Symfony event dispatcher directly - #10078

Merged
nickvergessen merged 3 commits into
masterfrom
techdebt/noid/prepare-28-event-dispatcher
Aug 4, 2023
Merged

fix: Stop using Symfony event dispatcher directly#10078
nickvergessen merged 3 commits into
masterfrom
techdebt/noid/prepare-28-event-dispatcher

Conversation

@nickvergessen

@nickvergessen nickvergessen commented Jul 28, 2023

Copy link
Copy Markdown
Member

☑️ Resolves

🖼️ Screenshots

🏚️ Before 🏡 After
Bildschirmfoto vom 2023-08-04 14-42-07 Bildschirmfoto vom 2023-08-04 14-41-49

🏁 Checklist

@nickvergessen nickvergessen added this to the 💜 Next Major (28) milestone Jul 28, 2023
@nickvergessen nickvergessen self-assigned this Jul 28, 2023
@nickvergessen
nickvergessen marked this pull request as draft July 28, 2023 16:28
@nickvergessen
nickvergessen force-pushed the techdebt/noid/prepare-28-event-dispatcher branch from 78cbfd3 to fc38ebe Compare July 28, 2023 16:48
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
nickvergessen force-pushed the techdebt/noid/prepare-28-event-dispatcher branch from fc38ebe to b57f32b Compare August 4, 2023 12:41
@nickvergessen
nickvergessen requested a review from Antreesy August 4, 2023 12:43
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
nickvergessen marked this pull request as ready for review August 4, 2023 12:48

@Antreesy Antreesy left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🦭

Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen

Copy link
Copy Markdown
Member Author

Guest app fails due to: nextcloud/guests#1045

@nickvergessen
nickvergessen disabled auto-merge August 4, 2023 13:55
@nickvergessen
nickvergessen merged commit c888ea9 into master Aug 4, 2023
@nickvergessen
nickvergessen deleted the techdebt/noid/prepare-28-event-dispatcher branch August 4, 2023 13:55
@nickvergessen nickvergessen mentioned this pull request Aug 4, 2023
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants