stats4sd / aec_portfolio

A proof of concept for the AEC Consortium Project Management / Assessment System
GNU General Public License v3.0
0 stars 0 forks source link

Fix Members permissions #214

Closed dave-mills closed 1 year ago

dave-mills commented 1 year ago

This PR fixes #203:

NOTE: This PR includes PR #210, because my local version started expecting the View Component class for x-help-text-link added in that PR. I recommend not reviewing this one until 210 is merged in, so then the changes made here will be much more clear.

ALSO NOTE: Institutional members can still 'delete' institutions in this version. That issue is fixed in #208.

dan-tang-ssd commented 1 year ago

I got latest code and execute RoleSeeder file.

Performed testing with positive result in local env. I think it is good to merge.


Screen shots for reference:

Login as institutional member

BEFORE this PR, no portfolio showed for institutional member image

AFTER this PR, portfolios showed for institutional member, with "SHOW INITIATIVES" button only image

AFTER this PR, Additional assessment criteria tab, it shows "VIEW" button only image


Login as institutional admin

AFTER this PR, portfolios showed for institutional admin, with "SHOW INITIATIVES", "EDIT", "REMOVE" buttons image

AFTER this PR, Additional assessment criteria tab, it shows "VIEW", "EDIT", "REMOVE" buttons image