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.

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

IMO and Partner Should not be able to edit title, label, etc (more in the ticket) for PAI derived from CAI #669

Closed
sethgold8 opened this issue May 16, 2018 · 5 comments

Comments

@sethgold8
Copy link
Contributor

sethgold8 commented May 16, 2018

Editing Partner Activity Indicator modal derived from CAI a should only allow Partner to add new locations or editing targets on locations. This happens for both Partner and IMO user

Right now IMO can edit Title, Comments, measurement specifications, Frequency of reporting, date of report, start date of report, labels (all), baseline and target on PAI even though they should be locked since they are derived from Cluster Activity

removed previous screen shot because it was the wrong one

@momimomo
Copy link
Contributor

I'm sorry, it was Rafał's component I think, it's going to take some time for me to go through all of it, so maybe let's assign him too.

@rkrupinski
Copy link
Contributor

@sethgold8 The UI also lets IMOs add partner activities / indicators. How's that coherent?

cc @n1207n

@sethgold8
Copy link
Contributor Author

@rkrupinski just the edit indicator modal of the photo attached above

@sethgold8
Copy link
Contributor Author

sethgold8 commented May 18, 2018

The main details above locations are not editable to so moving forward :)

Adding new ticket #679 though which is around not being able to edit baseline and in need for PAI derived from CAI.

@sethgold8 sethgold8 changed the title IMO should not be able to edit PAI Should not be able to edit PAI May 18, 2018
@sethgold8 sethgold8 changed the title Should not be able to edit PAI IMO and Partner Should not be able to edit title, label, etc (more in the ticket) for PAI derived from CAI May 19, 2018
@sethgold8
Copy link
Contributor Author

working :)

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

No branches or pull requests

4 participants