-
Notifications
You must be signed in to change notification settings - Fork 100
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
Show versions #886
Comments
We have a lot of dependencies. Instead of a modal, let's make a very simple full page that lists all of our |
I would not show all the dependency versions. Just the Gaia/Tendermint/Voyager versions. Which versions would a user be interested in? |
Nice :) |
What about the Tendermint version? |
For the design I like the 2nd one the most as it looks the cleanest. But the popup also looks nice and maybe putting the versions on the preferences page mixes up too much content? |
I think the popup is cleanest. But I wonder why the version shows 0.4.3. Good work. |
looks good! maybe the left side should ay "Version information" and the button on the right could say "show versions" - as the buttons tend to contain a verb and the left side (description) is usually more descriptive 😄 |
For the text, let's do:
If the SDK is following semver, we don't need to show the user the git hash--that is extra visual noise they don't need. |
What about the Tendermint version? |
@Fabo and I discussed Tendermint version and decided that because the tm version can be derived from Cosmos SDK version, and because there's no command available in gaiacli or gaiad right now to get the tendermint version that we skip it until it is requested. |
* trigger CI * release-1.0.23 * Update CHANGELOG.md Co-authored-by: Bitcoinera <[email protected]> Co-authored-by: Release Lunie Bot <[email protected]>
UI Version: 0.7.0
Description:
Following up on #767 we may want a modal that shows the versions currently used in Voyager or incorporate these informations on the preferences page.
The text was updated successfully, but these errors were encountered: