-
Notifications
You must be signed in to change notification settings - Fork 0
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
TM-6124-Projected-Vacancies-UI #3013
Conversation
… into TM-6124-Projected-Vacancies-Expose-Read
@@ -308,6 +309,7 @@ const ProjectedVacancyCard = (props) => { | |||
<PositionExpandableContent | |||
sections={sections} | |||
form={form} | |||
tempHideEdit={isAO} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the ticket asked to hide the edit button - not to disable it, added this in for ya!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good eye 👀
src/Constants/Menu.js
Outdated
icon: ' fa-keyboard-o', | ||
roles: [ | ||
'superuser', | ||
'bureau_user', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i think this should have been ao_user
... ill take care of it - tinkering with a few other things as well!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
surprised this worked for you locally?
there's a weird bug where you have to log out and back in twice locally to make sure your permissions come out correct.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@M00staff I may have to clear my cache when reloading.
Dual Merge:
Ticket