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

Dashboard: Allow control over whether it's possible to install Dashboard as a PWA #1616

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

joepavitt
Copy link
Collaborator

Description

  • Adds a new "Allow Installation" option to a Dashboard
  • Will toggle on/off the loading of the manifest.yml from the Dashboard, which in turn permits installation

Related Issue(s)

Closes #1588

@joepavitt joepavitt requested a review from knolleary February 11, 2025 15:14
@knolleary
Copy link
Member

I still see the <link rel="manifest"...> is present in dist/index.html when I build this branch.

Once I comment that out, then I no longer get the auth errors from #1588 - I think.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Error Accessing Dashboard with FlowFuse Authentication Enabled
2 participants