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

Create a banner saying that Namadillo is not updated #1491

Open
euharrison opened this issue Jan 10, 2025 · 3 comments
Open

Create a banner saying that Namadillo is not updated #1491

euharrison opened this issue Jan 10, 2025 · 3 comments

Comments

@euharrison
Copy link
Collaborator

euharrison commented Jan 10, 2025

We need a banner showing when Namadillo is not updated

We'll need a source of truth of which one is the last version. We can use the Github releases for that.
But it's a good idea to define a pattern, for example, only show the banner in case it's a minor update, not a patch
Otherwise they will need to update Namadillo almost everyday, and the banner will lose significance

Schermata_2025-01-10_alle_12 34 10

@h3ll0ofr1end
Copy link

Just tested the new component for version verification. The warning message is clear, as suggested by @ChrisHoltDesign, and includes division of the version into major, minor, and patch, comparing only for "minor" or "major" updates. @euharrison

Regarding the source for the latest version, I'm currently unable to retrieve the correct latest version from either https://api.github.com/repos/anoma/namada-interface/releases or https://api.github.com/repos/anoma/namada-interface/tags due to outdated values.
During testing, the fetch was performed from https://raw.githubusercontent.com/anoma/namada-interface/refs/heads/main/apps/namadillo/package.json.

@euharrison
Copy link
Collaborator Author

Oh, yes! The current releases tags are for the Keychain

Currently, it's ok to get from package.json

We have plans to split the namada-sdk and keychain into their own repository, but it could take some time to be done

@h3ll0ofr1end
Copy link

Great. We’ll send the PR soon then.

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

No branches or pull requests

2 participants