Skip to content

fix: Use getId() method for card ID retrieval#7139

Merged
grnd-alt merged 3 commits into
mainfrom
fix/6739
Jul 29, 2025
Merged

fix: Use getId() method for card ID retrieval#7139
grnd-alt merged 3 commits into
mainfrom
fix/6739

Conversation

@enjeck

@enjeck enjeck commented Jul 28, 2025

Copy link
Copy Markdown
Contributor

Resolves: #6739

Checklist

  • Code is properly formatted
  • Sign-off message is added to all commits
  • Tests (unit, integration, api and/or acceptance) are included
  • Documentation (manuals or wiki) has been updated or is not required

Signed-off-by: Enjeck C. <patrathewhiz@gmail.com>
@enjeck enjeck self-assigned this Jul 28, 2025
@github-project-automation github-project-automation Bot moved this to 🧭 Planning evaluation (don't pick) in 📝 Productivity team Jul 28, 2025
@luka-nextcloud

Copy link
Copy Markdown
Contributor

@enjeck Code change is ok.
My feedback: I think it's worth adding a Cypress test for creating a card from the dashboard.

@grnd-alt

Copy link
Copy Markdown
Member

I tested a bit, and for me creating a label in the dialog still results in the "labelId must be present..." error as reported in the issue. Just the same error for cardId is gone. I think we should look into that as well.

grnd-alt added 2 commits July 29, 2025 10:37
Signed-off-by: grnd-alt <github@belakkaf.net>
Signed-off-by: grnd-alt <github@belakkaf.net>
@grnd-alt grnd-alt merged commit 5744944 into main Jul 29, 2025
38 checks passed
@grnd-alt grnd-alt deleted the fix/6739 branch July 29, 2025 08:58
@github-project-automation github-project-automation Bot moved this from 🧭 Planning evaluation (don't pick) to ☑️ Done in 📝 Productivity team Jul 29, 2025
@grnd-alt

Copy link
Copy Markdown
Member

/backport to stable31

@grnd-alt

Copy link
Copy Markdown
Member

/backport to stable30

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

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

labelId must be provided and must be numeric

3 participants