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.

Sign up for GitHub

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

Jump to bottom

Remove users from a poll #854

Merged
merged 2 commits into from
Mar 2, 2020
Merged

Remove users from a poll #854

merged 2 commits into from
Mar 2, 2020

Conversation

dartcafe
Copy link
Collaborator

fix #736

@dartcafe dartcafe added this to the 1.4 milestone Feb 29, 2020
@dartcafe dartcafe requested a review from v1r0x February 29, 2020 19:39
@dartcafe dartcafe self-assigned this Feb 29, 2020
@dartcafe dartcafe changed the title delete user - initial commit Remove users from a poll Feb 29, 2020
v1r0x
v1r0x approved these changes Mar 2, 2020
View reviewed changes
Copy link
Collaborator

@v1r0x v1r0x left a comment

Choose a reason for hiding this comment

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

Looks good! 🐧 👍

@dartcafe
Copy link
Collaborator Author

dartcafe commented Mar 2, 2020

Looks good!

Not a pretty solution IMHO, but OK for now. Maybe someone has a better UX idea.

@dartcafe dartcafe merged commit 882067f into master Mar 2, 2020
@delete-merged-branch delete-merged-branch bot deleted the removeUser branch March 2, 2020 17:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@v1r0x v1r0x v1r0x approved these changes

Assignees

@dartcafe dartcafe

Labels
4. to release enhancement
Projects
None yet
Milestone
1.4
Development

Successfully merging this pull request may close these issues.

Remove Participants from vote
2 participants
@dartcafe @v1r0x