Skip to content

fix(cards): assign lables and users when creating a card via OCS - #8461

Merged
mejo- merged 1 commit into
mainfrom
fix/picker_new_card_attrs
Sep 29, 2026
Merged

mejo- merged 1 commit into
mainfrom
fix/picker_new_card_attrs

Conversation

@mejo-

@mejo- mejo- commented Sep 29, 2026

Copy link
Copy Markdown
Member

The OCS card create endpoint took labels and users but never applied them, because the assignment loops were commented out in commit 972c797.

Fixes: #8460

Assisted-by: ClaudeCode:claude-opus-5.5

Checklist

  • Code is properly formatted
  • Sign-off message is added to all commits

🤖 AI (if applicable)

  • The content of this PR was partly or fully generated using AI (N/A)

The OCS card create endpoint took `labels` and `users` but never
applied them, because the assignment loops were commented out in
commit 972c797.

Fixes: #8460

Signed-off-by: Jonas <jonas@freesources.org>
Assisted-by: ClaudeCode:claude-opus-5.5
@mejo-

mejo- commented Sep 29, 2026

Copy link
Copy Markdown
Member Author

/backport to stable35

@mejo-

mejo- commented Sep 29, 2026

Copy link
Copy Markdown
Member Author

PHPUnit test failures are unrelated. Here's a separate PR to fix them: #8462

@github-actions

Copy link
Copy Markdown
Contributor

🐢 Performance warning.
It looks like the query count of the integration tests increased with this PR.
Database query count is now 206535 was 198420 (+4.08%)
Please check your code again. If you added a new test this can be expected and the base value in tests/integration/base-query-count.txt can be increased.

@mejo-
mejo- merged commit c9e62b8 into main Sep 29, 2026
33 of 41 checks passed
@mejo-
mejo- deleted the fix/picker_new_card_attrs branch September 29, 2026 14:50
@mejo-

mejo- commented Sep 29, 2026

Copy link
Copy Markdown
Member Author

/backport to stable34

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: ☑️ Done

Development

Successfully merging this pull request may close these issues.

Deck card: Labels and assigned users are not transferred when creating via “/New Deck card” in a Collectives page

2 participants