tethysplatform / tethys

The Tethys Platform main Django website project repository.
http://tethysplatform.org/
BSD 2-Clause "Simplified" License
92 stars 49 forks source link

Add "access_app" permission to Proxy Apps #935

Closed sdc50 closed 1 year ago

sdc50 commented 1 year ago

Currently the "access_app" permission that allows the visibility of apps to be controlled (if the ENABLE_RESTRICTED_APP_ACCESS setting is True) only applies to TethysApp instances, and not to ProxyApps.

I would like the ability to assign the access_app permission to Groups for both types of app:

image

sdc50 commented 1 year ago

I'm adding a bug to this issue, because they are related. The JavaScript that handles hiding and showing specific app permissions is using the app name and trying to derive the app package. The display name and the package don't always correspond. We either need to use the app package in the selector rather than the name, or somehow embed the app package as a data field.

swainn commented 1 year ago

Merged in #936