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

@dartcafe dartcafe added the bug label Jan 22, 2022
@dartcafe dartcafe requested a review from juliusknorr January 22, 2022 08:29
Signed-off-by: dartcafe <github@dartcafe.de>
@dartcafe dartcafe force-pushed the fix/return-selector branch from 57c4b42 to a6516f0 Compare January 22, 2022 08:30
@juliusknorr
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.

@juliusknorr
Copy link
Member

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

@juliusknorr juliusknorr 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
@juliusknorr
Copy link
Member

/backport to stable23

@juliusknorr
Copy link
Member

/backport to stable22

@juliusknorr
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