Skip to content
This repository was archived by the owner on Mar 13, 2025. It is now read-only.

Hide Yes No buttons for Candidates list for user without access to update candidate status #38

Closed
SathyaJayabal opened this issue Dec 28, 2020 · 5 comments

Comments

@SathyaJayabal
Copy link
Collaborator

In cases where the user has only read permissions on the candidate, we must hide the Yes and No
buttons that update the candidate status
Example: Connect manager topcoder-platform/taas-apis#72
Screenshot 2020-12-28 at 10 54 57 AM
cc @maxceem

@maxceem
Copy link
Contributor

maxceem commented Jan 7, 2021

Let's keep this for the next release. At the moment we don't have a way to detect if the user has write permissions or only read permission client-side. I guess there would not be much harm if Connect Managers would see errors if they try to click these buttons as server would not let them update JobCandidates.

The best time to implement this is together with #14 because during implementing that issue, we would get information if user has write or only read permissions.

@maxceem maxceem added the enhancement New feature or request label Jan 7, 2021
@maxceem maxceem mentioned this issue Feb 13, 2021
5 tasks
@maxceem
Copy link
Contributor

maxceem commented Feb 23, 2021

@sandhiyakavi this is ready for QA as a part of #83

@sandhiyakavi
Copy link
Collaborator

Verified on Dev Env.Working as expected.

For members of the project.

image

For Connect manager

image

@sandhiyakavi
Copy link
Collaborator

Fixed: 🆗 in Production via milestone v1.5

Able to view Yes and No button for members of the project.
image

@lakshmiathreya Need to check Yes and No button not visible for Connect Manager (should not be a member of the project).

@lakshmiathreya
Copy link

This has been checked for Connect Manager - see #83 .

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants