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

Allow multiple nominations per users #446

Closed
wants to merge 1 commit into from
Closed

Allow multiple nominations per users #446

wants to merge 1 commit into from

Conversation

ks129
Copy link
Member

@ks129 ks129 commented Feb 20, 2021

Removes check that doesn't allow users have more than 1 active nomination. Updated docs and tests to match with this change. Closes #445

@ks129 ks129 added area: API Related to or causes API changes area: tests Related to `unittest` tests labels Feb 20, 2021
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling fd91a6f on ks123/multiple-nominations into bceaef8 on master.

@ks129
Copy link
Member Author

ks129 commented Feb 21, 2021

This will be resolved during the bigger refactor of nomination storage.

@ks129 ks129 closed this Feb 21, 2021
@ks129 ks129 deleted the ks123/multiple-nominations branch February 21, 2021 16:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
area: API Related to or causes API changes area: tests Related to `unittest` tests
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Allow many nominations per user
2 participants
@ks129 @coveralls