Restrict videocalling to full (level 2) access users #332
Conversation
| {this.props.t('physicianView:buttons.televideo')} | ||
| </a> | ||
| </div> | ||
| {this.props.authuser.profileStatus === 'level 2' && ( |
There was a problem hiding this comment.
this needs confirmation if "level 2" is at all used yet. cc @aviskarkc10 @sethuraj @rkchee
|
There seem to be some unresolved git conflicts here |
|
@cnagadya yeah, I realize... there's no conflicts when I change the base to master, so it seems they're out of sync again. in any case, @aviskarkc10 @sethuraj I'd like to confirm that there's indeed a "level 2" used for users at the moment or perhaps for now we want to give "level 1" users access to videocalls? https://trello.com/c/OH67aF5P/178-319-restrict-videocalling-to-full-level-2-access-users
|
gitjul
force-pushed
the
physician/televideo-access
branch
from
2d96765
to
782a8f6
Compare
|
conflicts resolved |
|
Great work!
… On Jun 30, 2020, at 6:16 AM, Julia Jacobs ***@***.***> wrote:
conflicts resolved
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.
|
|
@rkchee are you maybe able to answer the question about "level 2" already being used and thus ok to use as a condition for restricting the videocalls? |
Motivation
We don't want users that aren't fully verified physicians to be able to answer questions via videocall, this PR is to ensure that they can't use this functionality.
Description
With this change, the televideo button on the question board is only rendered for 'level 2' physicians.
Fixes #319
Type of change
New feature
Screenshots/Links
Before:
After: