Skip to content
This repository has been archived by the owner on Oct 5, 2024. It is now read-only.

Display user can vote state #311

Merged
merged 5 commits into from
Feb 18, 2018
Merged

Display user can vote state #311

merged 5 commits into from
Feb 18, 2018

Conversation

sklirg
Copy link
Member

@sklirg sklirg commented Feb 16, 2018

This renders a box with current user permissions. It will vary depending on permissions and voting state.

image

Close up:
image

Users who should be allowed to vote, but currently can not (canVote is false) gets a warning symbol:
image

If you are not allowed to vote at all, some irrelevant information is removed:
image

It also works for managers and admins:
image

@codecov
Copy link

codecov bot commented Feb 16, 2018

Codecov Report

Merging #311 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #311   +/-   ##
=======================================
  Coverage   87.66%   87.66%           
=======================================
  Files          21       21           
  Lines         762      762           
=======================================
  Hits          668      668           
  Misses         94       94
Impacted Files Coverage Δ
server/channels/connection.js 100% <ø> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update fe77c9e...eba5a6c. Read the comment docs.

@sklirg sklirg merged commit d483ce9 into master Feb 18, 2018
@sklirg sklirg deleted the feat/user-canVote-state branch February 18, 2018 16:14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants