Skip to content

refactor(workflowengine): Check if class is correct - #56630

Merged
AndyScherzinger merged 1 commit into
masterfrom
carl/workflowengine-hardening
Dec 3, 2025
Merged

refactor(workflowengine): Check if class is correct#56630
AndyScherzinger merged 1 commit into
masterfrom
carl/workflowengine-hardening

Conversation

@CarlSchwan

@CarlSchwanCarlSchwan commented Nov 24, 2025

Copy link
Copy Markdown
Member

Summary

TODO

  • ...

Checklist

@CarlSchwanCarlSchwan added this to the Nextcloud 33 milestone Nov 24, 2025
@CarlSchwanCarlSchwan self-assigned this Nov 24, 2025
@CarlSchwanCarlSchwan added the 3. to review Waiting for reviews label Nov 24, 2025
github-advanced-security[bot]
github-advanced-securityAI found potential problems Nov 24, 2025
View reviewed changes
Comment threadapps/workflowengine/lib/Manager.php Fixed
Comment threadapps/workflowengine/lib/Manager.php Fixed
@CarlSchwan
CarlSchwanforce-pushed the carl/workflowengine-hardening branch 2 times, most recently from 03ef666 to ffe9d83CompareNovember 25, 2025 08:53
@CarlSchwan
refactor(workflowengine): Check if class is correct
7541afa
Signed-off-by: Carl Schwan <carl.schwan@nextcloud.com>
@CarlSchwan
CarlSchwanforce-pushed the carl/workflowengine-hardening branch from ffe9d83 to 7541afaCompareNovember 26, 2025 09:17
@CarlSchwan
CarlSchwan requested a review from a team as a code ownerNovember 26, 2025 09:17
@CarlSchwan
CarlSchwan requested review from icewind1991, leftybournes, salmart-dev and yemkareems and removed request for a teamNovember 26, 2025 09:17
github-advanced-security[bot]
github-advanced-securityAI found potential problems Nov 26, 2025
View reviewed changes
Comment threadapps/workflowengine/lib/Manager.php Dismissed
Comment threadapps/workflowengine/lib/Manager.php Dismissed
Comment threadapps/workflowengine/lib/Manager.php Dismissed
@AndyScherzinger
AndyScherzinger merged commit 5b1d928 into masterDec 3, 2025
193 of 204 checks passed
@AndyScherzinger
AndyScherzinger deleted the carl/workflowengine-hardening branch December 3, 2025 15:21
@CarlSchwan

CarlSchwan commented Dec 3, 2025

Copy link
Copy Markdown
MemberAuthor

/backport to stable32

@CarlSchwan

CarlSchwan commented Dec 3, 2025

Copy link
Copy Markdown
MemberAuthor

/backport to stable31

This was referenced Dec 3, 2025
Merged
Merged
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ChristophWurstChristophWurstChristophWurst approved these changes
@blizzzblizzzblizzz approved these changes
@come-nccome-nccome-nc approved these changes
@nickvergessennickvergessenAwaiting requested review from nickvergessennickvergessen is a code owner
@icewind1991icewind1991Awaiting requested review from icewind1991icewind1991 is a code owner automatically assigned from nextcloud/server-backend
@salmart-devsalmart-devAwaiting requested review from salmart-devsalmart-dev is a code owner automatically assigned from nextcloud/server-backend
@yemkareemsyemkareemsAwaiting requested review from yemkareemsyemkareems is a code owner automatically assigned from nextcloud/server-backend
@leftybournesleftybournesAwaiting requested review from leftybournesleftybournes is a code owner automatically assigned from nextcloud/server-backend

Assignees

@CarlSchwanCarlSchwan

Labels

3. to reviewWaiting for reviews

Projects

None yet

Milestone

Nextcloud 33

Development

Successfully merging this pull request may close these issues.

6 participants

@CarlSchwan@ChristophWurst@blizzz@github-advanced-security@come-nc@AndyScherzinger