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

For BLOB downloads make sure the file is not preview-able before triggering the download #3472

Merged
merged 1 commit into from
Oct 24, 2024

Conversation

miasma13
Copy link
Contributor

Task/Issue URL: https://app.asana.com/0/414709148257752/1207504349745342/f

Description:
When wallet passes are served as BLOB type downloads they just trigger the download prompt instead of directly open their preview allowing adding them to Wallet app.

Steps to test this PR:

  1. Open http://lu.ma/
  2. (register with email - not sure if necessary)
  3. Find a free event
  4. Join the event
  5. Tap the small wallet icon
  6. Wallet pass should open directly in modal preview allowing adding it to wallet

Simulator Screenshot - iPhone 15 - 2024-10-23 at 19 27 37

Definition of Done (Internal Only):

Copy Testing:

  • Use of correct apostrophes in new copy, ie rather than '

Internal references:

Software Engineering Expectations
Technical Design Template

Copy link
Contributor

@brindy brindy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Tested some general download scenarios too.

@miasma13 miasma13 merged commit a0be7f8 into main Oct 24, 2024
16 checks passed
@miasma13 miasma13 deleted the michal/fix-blob-wallet-pass branch October 24, 2024 11:07
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.

2 participants