cessda / cessda.cvs.two

Apache License 2.0
0 stars 2 forks source link

Inconsistency with User Roles #466

Closed cessda-bitbucket-importer closed 1 year ago

cessda-bitbucket-importer commented 1 year ago

Original report on BitBucket by Joshua Tetteh Ocansey (GitHub: Joshocan).


How did we miss this!!

No user can ADD a CV on the Production instance.

We need to address this urgently before the next release

cessda-bitbucket-importer commented 1 year ago

Original comment by Joshua Tetteh Ocansey (GitHub: Joshocan).


@‌Martin Bobak @pakoselo @Stifo Is this something that can be fixed ASAP to allow @‌dolinarm to proceed with testing other functionalities deployed earlier?

cessda-bitbucket-importer commented 1 year ago

Original comment by Martin Šeleng (GitHub: pakoselo).


The CV button doesn't work for at least we start work on CV development. It doesn't work for iisas_guest user at dev and staging as well. But if you click on edit menu button and then click on create CV it magically works. I was thinking that the problem was with roles of iisas_guest account and it doesn't bother me anymore. Locally it's working for all of us, I guess. So the problem is maybe somewhere else.

Martin

cessda-bitbucket-importer commented 1 year ago

Original comment by Joshua Tetteh Ocansey (GitHub: Joshocan).


@pakoselo @‌dolinarm I found the problem now. You must at least have ROLE_USER in addition to any of the admin roles ( such as ROLE_ADMIN, ROLE_ADMIN_CONTENT, ROLE_ADMIN_TECHNICAL) to be able to “Add CV”.

This is strange behavior we need to fix.

By the way @‌dolinarm I have updated your role on both Production and on Dev. You should be able to create or add a CV now.

cessda-bitbucket-importer commented 1 year ago

Original comment by Martin Šeleng (GitHub: pakoselo).


@‌dolinarm @‌Martin Bobak @Joshocan is right, the problem is with USER_ROLE, but it shouldn’t be there anymore. I correct the problem in the code on my local instance and I will talk with Martin to integrate it in his changes as he is responsible for implementation of the new roles mechanism.

cessda-bitbucket-importer commented 1 year ago

Original comment by Joshua Tetteh Ocansey (GitHub: Joshocan).


Great @pakoselo . It means we can get it through this release cycle. Thanks

cessda-bitbucket-importer commented 1 year ago

Original comment by Maja Dolinar.


This is resolved in version 3.0.0