Skip to content

Use the new Events in Flow - #18535

Merged
blizzz merged 1 commit into
masterfrom
enh/flow/newDispatcher
Jan 7, 2020
Merged

Use the new Events in Flow#18535
blizzz merged 1 commit into
masterfrom
enh/flow/newDispatcher

Conversation

@rullzer

Copy link
Copy Markdown
Member

IMO the new events are much cleaner.
But lets be pragmatic and just emit both now and kill the legacy stuff for 19.

@rullzerrullzer added enhancement 3. to review Waiting for reviews labels Dec 22, 2019
@rullzerrullzer added this to the Nextcloud 18 milestone Dec 22, 2019
@rullzerrullzer mentioned this pull request Dec 23, 2019
18 tasks

@juliusknorrjuliusknorr left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

PHPDoc is needed for OCP\WorkflowEngine\Events\RegisterOperationsEvent::RegisterOperationsEvent
15 | PHPDoc is needed for OCP\WorkflowEngine\Events\RegisterOperationsEvent::__construct
16 | PHPDoc is needed for OCP\WorkflowEngine\Events\RegisterOperationsEvent::registerOperation
17 | PHPDoc is needed for OCP\WorkflowEngine\Events\RegisterChecksEvent::RegisterChecksEvent
18 | PHPDoc is needed for OCP\WorkflowEngine\Events\RegisterChecksEvent::__construct
19 | PHPDoc is needed for OCP\WorkflowEngine\Events\RegisterChecksEvent::registerCheck
20 | PHPDoc is needed for OCP\WorkflowEngine\Events\RegisterEntitiesEvent::RegisterEntitiesEvent
21 | PHPDoc is needed for OCP\WorkflowEngine\Events\RegisterEntitiesEvent::__construct
22 | PHPDoc is needed for OCP\WorkflowEngine\Events\RegisterEntitiesEvent::registerEntity
23

@rullzerrullzer mentioned this pull request Dec 27, 2019
13 tasks
@rullzer
rullzerforce-pushed the enh/flow/newDispatcher branch from 7fecb05 to 39a597fCompareDecember 27, 2019 13:35
@rullzer

Copy link
Copy Markdown
MemberAuthor

Fixed

@gary-kim

Copy link
Copy Markdown
Member

Looks like the tests need fixing :)
https://drone.nextcloud.com/nextcloud/server/24680/16/5

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
@rullzer
rullzerforce-pushed the enh/flow/newDispatcher branch from 39a597f to 25d4f32CompareJanuary 2, 2020 15:40
@rullzerrullzer mentioned this pull request Jan 2, 2020
1 task
@rullzer

Copy link
Copy Markdown
MemberAuthor

And finally all fixed.

@blizzz

Copy link
Copy Markdown
Member

i'll review tomorrow

@blizzzblizzz left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

looks good and works

@blizzz
blizzz merged commit 55fd157 into masterJan 7, 2020
@blizzz
blizzz deleted the enh/flow/newDispatcher branch January 7, 2020 08:58
MorrisJobke added a commit to nextcloud/documentation that referenced this pull request Aug 10, 2020
* LoadAdditionalScripts (@rullzer) - nextcloud/server#16641
* LoadViewerEvent (@skjnldsv) - nextcloud/viewer#271
* RegisterDirectEditorEvent (@juliushaertl) - nextcloud/server#17625
* typed events for files scanner (@ChristophWurst) - nextcloud/server#18351
* typed events for group mangement (@ChristophWurst) - nextcloud/server#18350
* AddContentSecurityPolicyEvent (@rullzer) - nextcloud/server#15730
* UserLiveStatusEvent (@georgehrke) - nextcloud/server#21186
* password_policy events (@ChristophWurst) - nextcloud/server#18019
* AddFeaturePolicyEvent (@rullzer) - nextcloud/server#16613
* ShareCreatedEvent (@rullzer) - nextcloud/server#18384
* LoadSettingsScriptsEvent (@blizzz) - nextcloud/server#21475
* flow events (@rullzer) - nextcloud/server#18535
Signed-off-by: Morris Jobke <hey@morrisjobke.de>
backportbot-nextcloudBot pushed a commit to nextcloud/documentation that referenced this pull request Aug 10, 2020
* LoadAdditionalScripts (@rullzer) - nextcloud/server#16641
* LoadViewerEvent (@skjnldsv) - nextcloud/viewer#271
* RegisterDirectEditorEvent (@juliushaertl) - nextcloud/server#17625
* typed events for files scanner (@ChristophWurst) - nextcloud/server#18351
* typed events for group mangement (@ChristophWurst) - nextcloud/server#18350
* AddContentSecurityPolicyEvent (@rullzer) - nextcloud/server#15730
* UserLiveStatusEvent (@georgehrke) - nextcloud/server#21186
* password_policy events (@ChristophWurst) - nextcloud/server#18019
* AddFeaturePolicyEvent (@rullzer) - nextcloud/server#16613
* ShareCreatedEvent (@rullzer) - nextcloud/server#18384
* LoadSettingsScriptsEvent (@blizzz) - nextcloud/server#21475
* flow events (@rullzer) - nextcloud/server#18535
Signed-off-by: Morris Jobke <hey@morrisjobke.de>
MorrisJobke added a commit to nextcloud/documentation that referenced this pull request Aug 10, 2020
* LoadAdditionalScripts (@rullzer) - nextcloud/server#16641
* LoadViewerEvent (@skjnldsv) - nextcloud/viewer#271
* RegisterDirectEditorEvent (@juliushaertl) - nextcloud/server#17625
* typed events for files scanner (@ChristophWurst) - nextcloud/server#18351
* typed events for group mangement (@ChristophWurst) - nextcloud/server#18350
* AddContentSecurityPolicyEvent (@rullzer) - nextcloud/server#15730
* UserLiveStatusEvent (@georgehrke) - nextcloud/server#21186
* password_policy events (@ChristophWurst) - nextcloud/server#18019
* AddFeaturePolicyEvent (@rullzer) - nextcloud/server#16613
* ShareCreatedEvent (@rullzer) - nextcloud/server#18384
* LoadSettingsScriptsEvent (@blizzz) - nextcloud/server#21475
* flow events (@rullzer) - nextcloud/server#18535
Signed-off-by: Morris Jobke <hey@morrisjobke.de>
MorrisJobke added a commit to nextcloud/documentation that referenced this pull request Aug 10, 2020
* LoadAdditionalScripts (@rullzer) - nextcloud/server#16641
* LoadViewerEvent (@skjnldsv) - nextcloud/viewer#271
* RegisterDirectEditorEvent (@juliushaertl) - nextcloud/server#17625
* typed events for files scanner (@ChristophWurst) - nextcloud/server#18351
* typed events for group mangement (@ChristophWurst) - nextcloud/server#18350
* AddContentSecurityPolicyEvent (@rullzer) - nextcloud/server#15730
* UserLiveStatusEvent (@georgehrke) - nextcloud/server#21186
* password_policy events (@ChristophWurst) - nextcloud/server#18019
* AddFeaturePolicyEvent (@rullzer) - nextcloud/server#16613
* ShareCreatedEvent (@rullzer) - nextcloud/server#18384
* LoadSettingsScriptsEvent (@blizzz) - nextcloud/server#21475
* flow events (@rullzer) - nextcloud/server#18535
Signed-off-by: Morris Jobke <hey@morrisjobke.de>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@rullzer@gary-kim@blizzz@juliusknorr