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

Wallet Links #275

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

lastmeta
Copy link

Currently the wallet link for Android is dead. I added the link for Moontree's Android wallet in it's place. I also replaced the ios wallet link with the Moontree's Wallet link for ios.

@Tylerlhess
Copy link

Tylerlhess commented May 3, 2024

I am split on this. I like the idea of having the link fixed. I don't like that this is basically endorsing Moontree wallet as the community accepted default. I would feel better If all of the asset supported apps would be listed with the download link linking to a generic search for RVN wallets. That being said I also see value in having the different app links there but having it more like a table. Having the app logo in column 1, Ios link in column2, and Android link to it in column 3. But making sure to do that for every asset aware app wallet. It would also be good to have for the 3rd party apps which is honestly where the Moontree wallet should be linked unless we are to make the other changes.

Also I think it would be great to have this page / these links checked on a regular basis. Checking for things such as that the links work and that the apps/wallets listed here are regularly update. (Oh! We could list the apps in order of latest release date newest first. Software maintenance is a much better listing order than alphabetical). Would checking on releasees and proper ordering and linkage be something you would want to take on? I have no power or sway either way just trying to lead the herd of cats in a coherent direction.

Copy link
Collaborator

@jeroz1 jeroz1 left a comment

Choose a reason for hiding this comment

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

The android link seems to be broken

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.

3 participants