Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

return the selector for collections #3552

Merged
merged 1 commit into from
Mar 4, 2022
Merged

Conversation

dartcafe
Copy link
Contributor

@dartcafe dartcafe commented Jan 22, 2022

I stumbled over it, when adding collections to Polls.

  • Target version: master

Summary

Avoids TypeError: can't access property "then", e.action() is undefined when adding a board or card as collection

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: dartcafe <github@dartcafe.de>
@juliushaertl
Copy link
Member

Nice catch, thanks for the pull request 👍

@dartcafe
Copy link
Contributor Author

dartcafe commented Mar 3, 2022

@juliushaertl Just found this unmerged. Was I expected to merge this? If yes, I'll rebase with a new review.

@juliushaertl
Copy link
Member

Sorry, totally slipped through here. Yes, this is good to get in of course :) Thanks for your ping.

@juliushaertl juliushaertl merged commit 9bbef1f into master Mar 4, 2022
@delete-merged-branch delete-merged-branch bot deleted the fix/return-selector branch March 4, 2022 08:53
@juliushaertl
Copy link
Member

/backport to stable23

@juliushaertl
Copy link
Member

/backport to stable22

@juliushaertl
Copy link
Member

/backport to stable21

@backportbot-nextcloud
Copy link

The backport to stable22 failed. Please do this backport manually.

@backportbot-nextcloud
Copy link

The backport to stable21 failed. Please do this backport manually.

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.

2 participants