You need to sign in or sign up before continuing.
[engine] Dynamic PWA manifest generation minds#4645
Ticket(s) / Related Merge Requests
- Issue card: minds#4645 (closed)
- Helm charts: helm-charts!164 (merged)
- Minds root repo: minds!216 (merged)
Summary of Changes
Generate PWA manifest dynamically for tenants and main Minds site.
Testing Considerations
Manifest files
WARNING - Just so you don't panic, these MAY download the web manifest file rather than open it in a browser depending on which browser you are using. Just check the file's contents, and feel free to delete it when you are done.
Check contents of the below files are relevant to Minds / The bound tenant network:
-
https://ben.oke.minds.io/manifest.webmanifest -
https://feat-dynamic-manifest-m4645.oke.minds.io/manifest.webmanifest
PWA
Check the below links in a browser with no cache (I used Chrome) - you will need to uninstall the Minds app to see the root Minds one.
-
https://ben.oke.minds.io/ - you get prompted to install the tenant app (which installs) -
https://feat-dynamic-manifest-m4645.oke.minds.io/manifest.webmanifest - you get prompted to install the Minds app from the app store.
Deployment Considerations
Make sure to merge all repos including helm and nginx!
Regression Scope
PWA
Platform Affected (web, mobile, etc)
Web, PWA.
Developer Testing Completed
Manual, unit (phpspec)
Screenshots / Screen Recording
Does this impact
-
Localization -
Dark/light mode -
Guest mode
Definition of Done Checklist
-
The Acceptance Criteria has been met -
Code is tested: Testing includes unit/spec, E2E/automated and manual testing -
Merge requests description has been filled out
Edited by Ben