BiologicalRecordsCentre / UKBMS-online

Issue tracking for UKBMS online recording site
2 stars 0 forks source link

Masquerading #303

Closed RachaelConway closed 1 year ago

RachaelConway commented 1 year ago

@BirenRathod @IanMiddlebrook

Opening thread just for masquerading as suggested by @DavidRoy

@BirenRathod It seems to be as Ian says, a problem with the Branch Coordinator role. It isn't because I didn't unmasquerade. I don't seem to have the other masquerade entry you described

BirenRathod commented 1 year ago

@RachaelConway Thanks for opening this issue. you already got admin role. Are you trying to resolve this for people who has got "Branch coordinator role" ?

RachaelConway commented 1 year ago

@RachaelConway Thanks for opening this issue. you already got admin role. Are you trying to resolve this for people who has got "Branch coordinator role" ?

Hi @BirenRathod I'm referring to the online users, after a search and select - if they are a normal user then the masquerade drop down is present, however if they are a branch coordinator the masquerade drop down is not present. Please see both roles as below

image image
BirenRathod commented 1 year ago

@RachaelConway That is interesting, I didn't see that. I just enabled permission for authenticated users but permission doesn't consider role "branch coordinator" as authenticated users. I have implemented that.

DavidRoy commented 1 year ago

@BirenRathod normal users should not be able to masquerade as others

BirenRathod commented 1 year ago

@DavidRoy masquerading option only appears for "admin" user, so admin users can masquerade as "authenticated users" but not vice versa.

IanMiddlebrook commented 1 year ago

@BirenRathod Now not able to masquerade for any user

BirenRathod commented 1 year ago

@IanMiddlebrook somebody else is changing the permission setting which I did yesterday. I reverted it back now. Can you check it again, please?

IanMiddlebrook commented 1 year ago

Thanks @BirenRathod that's back to normal.

BirenRathod commented 1 year ago

@IanMiddlebrook & @RachaelConway Please close this if you satisfy.