Added gatekeeping dialogs for PWA install requirement
#71
Open
jose
wants to merge 1 commits from friend-tech-inspired-pwa-dialog
into master
pull from: friend-tech-inspired-pwa-dialog
merge into: trent_larson:master
trent_larson:adjust-note
trent_larson:button-visual-enhancement
trent_larson:bvc-shortcut
trent_larson:claim-view-improvements
trent_larson:cleanup-and-qr-code
trent_larson:contact-amounts-ui-improvements
trent_larson:design-tweaks-2023-12
trent_larson:experimental_plugin
trent_larson:fix-contact-totals
trent_larson:gifted-camera-improvements
trent_larson:home-infinite
trent_larson:home-view-notification-improvements
trent_larson:identity-switcher
trent_larson:increment-derived
trent_larson:jdenticon-entity-photos
trent_larson:many-misc
trent_larson:master
trent_larson:misc2
trent_larson:miscellany
trent_larson:more-smalls
trent_larson:move-id-switch
trent_larson:new-web-push
trent_larson:no-accounts-in-memory
trent_larson:nostr
trent_larson:notification-request-permission-dialog
trent_larson:notify-time
trent_larson:notiwind-alert
trent_larson:offer-edit
trent_larson:openssl
trent_larson:passkey
trent_larson:passkey-cache
trent_larson:photo-ratio
trent_larson:photo-reverse
trent_larson:photo-upload
trent_larson:plan-loc
trent_larson:playwright-pwa-install-test
trent_larson:polish-ui-project-view
trent_larson:profile-pic
trent_larson:project-gives
trent_larson:project-map-link
trent_larson:projects-view-improvements
trent_larson:qr-reader
trent_larson:qr-reader-rebased
trent_larson:quicknav-component
trent_larson:remove-old-alerts
trent_larson:search-bbox
trent_larson:searching
trent_larson:seed-backup-view-improvements
trent_larson:service-worker-final
trent_larson:similarify-search
trent_larson:simple-signer
trent_larson:starred-projects
trent_larson:sw-cleanup
trent_larson:tmp
trent_larson:try-cypress
trent_larson:tweaks
trent_larson:ui-fix
trent_larson:ui-fixes-2024-03
trent_larson:update-api-server
trent_larson:v-onboarding-2024-04
trent_larson:vite-version
trent_larson:web-push-permissions
trent_larson:why-migrate-fail
trent_larson:world-fix
trent_larson:yml-fixes
Reviewers
Request review
No reviewers
Labels
Apply labels
Clear labels
No items
No Label
Milestone
Set milestone
Clear milestone
No items
No Milestone
Assignees
Assign users
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.
No due date set.
Dependencies
This pull request currently doesn't have any dependencies.
Reference in new issue
There is no content yet.
Delete Branch 'friend-tech-inspired-pwa-dialog'
Deleting a branch is permanent. It CANNOT be undone. Continue?
No
Yes
This makes sense. I'd like to understand how the pwa-install-gate-* flags are set.
Note that we'd like to make this an optional step, such that they can still interact even if they're not installed on the desktop. Is this allowed with this setup?
Jose says: "maybe we can change the wording on the dialog to indicate a "soft requirement", and only trigger it where appropriate"
The flags were not straightforward. I ended up inserting some UI for this in the HomePage here.
You can see that it's a check for notification support, because I found that there was no trustworthy flags to use to see if the app was installed, at least on iOS. I have some potential Android checks documented here.
I'd also like to put this front-and-center, but not in a way that they have to close a pop-up to get around it. Maybe there's a better way.
So... I'd love to see better approaches. If there is a different workflow & set of logic that works better, let's discuss it.