Skip to content

fix(workflowengine): Fix list of checks being a list instead of array…#58664

Merged
nickvergessen merged 1 commit intomasterfrom
bugfix/noid/workflowengine-fix-checks-list
Mar 5, 2026
Merged

fix(workflowengine): Fix list of checks being a list instead of array…#58664
nickvergessen merged 1 commit intomasterfrom
bugfix/noid/workflowengine-fix-checks-list

Conversation

@nickvergessen
Copy link
Member

@nickvergessen nickvergessen commented Mar 2, 2026

…<int, …>

Steps

  1. Create a flow
grafik
  1. Reload the page
  2. Seems psalm was right and I was wrong 🙈
  3. Regression from feat(workflowengine): Add openapi for workflow engine #58278

Expected

Workflow shows

Actual

Error about some() in console
grafik

Checklist

AI (if applicable)

  • The content of this PR was partly or fully generated using AI

…<int, …>

Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen nickvergessen added this to the Nextcloud 34 milestone Mar 2, 2026
@nickvergessen nickvergessen requested a review from ShGKme March 2, 2026 13:20
@nickvergessen nickvergessen self-assigned this Mar 2, 2026
@nickvergessen nickvergessen requested a review from blizzz as a code owner March 2, 2026 13:20
Copy link
Contributor

@szaimen szaimen left a comment

Choose a reason for hiding this comment

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

🐘

@nickvergessen nickvergessen merged commit 85b0dcc into master Mar 5, 2026
196 of 208 checks passed
@nickvergessen nickvergessen deleted the bugfix/noid/workflowengine-fix-checks-list branch March 5, 2026 09:49
Sign up for free to 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