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

add android and ios links to the cryptopower card on /wallets #1167

Closed
wants to merge 2 commits into from

Conversation

dreacot
Copy link
Contributor

@dreacot dreacot commented Feb 17, 2024

Screenshot 2024-03-16 at 6 05 49 PM

@dreacot dreacot changed the title add android and ios links to the cryptopower card add android and ios links to the cryptopower card on /wallets Feb 17, 2024
@xaur
Copy link
Contributor

xaur commented Feb 21, 2024

Small thing but while we're at it, how about we promote the US link as the main one?

Link in this PR: https://apps.apple.com/ca/app/cryptopower-wallet/id6472668308

US version: https://apps.apple.com/us/app/cryptopower-wallet/id6472668308

Looks like the reviews and rating are tied to the region. So we miss that 1 review in the CA link. But if we check similar apps, the US link dominates:

@xaur
Copy link
Contributor

xaur commented Feb 21, 2024

@dreacot let's also update the "Downloads and release notes" to link v1.1.0. And maybe add a third download option titled "APK" or "Android APK" and linking to the APK file in the v1.1.0 release.

@dreacot
Copy link
Contributor Author

dreacot commented Mar 16, 2024

@xaur the changes have been effected

@xaur
Copy link
Contributor

xaur commented Mar 16, 2024

Looks great thank you!

@xaur
Copy link
Contributor

xaur commented Apr 5, 2024

@xaur
Copy link
Contributor

xaur commented Apr 11, 2024

@dajohi @jholdstock we can merge this as is and I'll make a small follow-up PR to update the two links to v1.1.2 to speed this up a bit. I can't push to this PR.

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