Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow to configure the available menu entries #1506

Open
csware opened this issue Nov 26, 2022 · 2 comments
Open

Allow to configure the available menu entries #1506

csware opened this issue Nov 26, 2022 · 2 comments
Labels
0. Needs triage Pending approval or rejection. This issue is pending approval. enhancement New feature or request

Comments

@csware
Copy link

csware commented Nov 26, 2022

Is your feature request related to a problem? Please describe.
The menu of the photo app contains menu options for "faces" and "maps" - even when the apps are not installed (or I as a admin never want to install these).

Describe the solution you'd like
Automatically hide the options when the apps are not installed or make it configurable to hide these entries.

@csware csware added enhancement New feature or request 0. Needs triage Pending approval or rejection. This issue is pending approval. labels Nov 26, 2022
@bpmartin20
Copy link

+1

@joshtrichards
Copy link
Member

joshtrichards commented Jul 1, 2023

They are hidden per #1155 and #343 if the apps aren't installed, but there is an exception: if you're logged in as an administrator. I'm not sure why that exception is made.

photos/src/Photos.vue

Lines 161 to 167 in 1b2ceb6

showLocationMenuEntry: getCurrentUser() === null
? false
: getCurrentUser().isAdmin || isMapsInstalled,
showPeopleMenuEntry: getCurrentUser() === null
? false
: getCurrentUser().isAdmin || isRecognizeInstalled,

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0. Needs triage Pending approval or rejection. This issue is pending approval. enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants