Moonshine-IDE / Super.Human.Portal

Portal interface to show documentation for DominoVagrant and Super.Human.Installer
Other
0 stars 1 forks source link

Allow Non-Administrators to View Genesis Applications #59

Open piotrzarzycki21 opened 2 weeks ago

piotrzarzycki21 commented 2 weeks ago

From @JustinProminic

I asked Triston to run some tests, and he when using Super.Human.Portal as not an administrator is unable to see the other applications to install.

I think we should allow non-administrators to view, but not install, the applications.

Currently the Genesis Apps and Installed Apps are hidden if the user does not have the Administrator role:

image

We want to make the applications visible to the other users so that if they see something they need, they can ask an administrator to install it. For now, I'd like to show them something like this:

image

So, users without the Administrator role would be able to see

However, these features should only be available if the user has the "Administrator" role.

We discussed changing the Install button to make it send an email to the administrators instead, but this would be a bigger change on the backend.

I will need to update the GenesisRead to allow non-Administrator access, but I ran out of time for this today.

Note that this is a change to the roles we decided on for #43 https://github.com/Moonshine-IDE/Super.Human.Portal/issues/43#issuecomment-1821821786 https://github.com/Moonshine-IDE/Super.Human.Portal/issues/43#issuecomment-1843595625 https://github.com/Moonshine-IDE/Super.Human.Portal/blob/2f7ede5e98da2ba606115b5bb01c1cd8135392b6/Super.Human.Portal_Agents/src/test/groovy/auth/SimpleRoleSecuritySpec.groovy#L157-L178

piotrzarzycki21 commented 2 weeks ago

@JoelProminic this is issue for that quick update. If you won't add details today I will do this tomorrow.

piotrzarzycki21 commented 2 weeks ago

@JoelProminic I have pushed changes for this issue. I believe you need remove restriction for retrieving Genesis Apps and Installed application on your sight, because now we are getting error during that operation. Newest build should have my changes.

Changes has been pushed to main and #56 branch.

JoelProminic commented 2 weeks ago

I made an update to the features/issue_56_tile_view_getting_started branch (we can cherry pick this to main if desired). With the update, this is mostly working. I see a problems with the Install button, though (which you wouldn't have been able to test).

Feature Administrator Not Administrator
Genesis Apps Visible Visible
Genesis Apps > Additional Directories Visible Hidden
Installed Apps Visible Visible
Install button Original label. BUG: Install button was enabled even if no app was selected Visible, label updated. BUG: enabled, so we get a permissions error on the agent.

I noticed that the Genesis Apps is not highlighted when it is selected in the sidebar. This seems to be an existing issue - I suspect this is because it is a category/folder.

piotrzarzycki21 commented 2 weeks ago

Install button Original label. BUG: Install button was enabled even if no app was selected Visible, label updated. BUG: enabled, so we get a permissions error on the agent.

I have pushed fix to main and my branch for that part.

piotrzarzycki21 commented 2 weeks ago

I noticed that the Genesis Apps is not highlighted when it is selected in the sidebar. This seems to be an existing issue - I suspect this is because it is a category/folder.

@JoelProminic I have fixed this one as well. It's been pushed to main and to branch with tile view.