Skip to content

Vaulta Rebrand and New Contract Support #35

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 48 commits into
base: develop
Choose a base branch
from
Open

Vaulta Rebrand and New Contract Support #35

wants to merge 48 commits into from

Conversation

ericpassmore
Copy link

Summary

EOS has rebranded to Vaulta. This update changes the brand name and logos. App code changes were required. Tests have been added and re-organized to support old and new contract names.

Security Review

Small code changes may require security review

Checklist

  • App update process has been followed
  • Target branch is develop
  • Application version has been bumped

Dependency

Depends on App-Database Update LedgerHQ/ledger-app-database#353

Details

EOS is rebranding to Vaulta, with a new ticker symbol A.

  • Update logos to match new branding
  • Update to AppName to Vaulta branding name
  • Remove references to unsupported fairy wallet
  • Refactor tests to distinguish legacy eosio tests and core.vaulta tests
  • Add additional test cases for A token and core.vaulta contract
  • Added wumpas tests with bad transfer actions, and validated they came across as unknown actions
  • Update Golden Images for testing. Includes new subdirectory structure for Golden Images for better isolation of eosio and vaulta transactions.
  • Upgraded tests to ledgered.Devices
  • Updated pylint config to accept more branches due to added test cases
  • Updated readme
  • Removed nanos from .toml
  • Upgraded Action in tests to accept straight hex-encoded data when provided.

ericpassmore and others added 30 commits February 7, 2025 10:49
@ericpassmore
Merge pull request #45 from eosnetworkfoundation/Force-Ledger-HQ-Feb-…
d8ff388 
…2025

Force Merg Ledger HQ Feb 2025
@ericpassmore
reworking docs for clarity with MacOS up
9093699
@ericpassmore
added python venv to gitignore
51c5aa6
@ericpassmore
updated documentation for testing ledger
b172c3e
@ericpassmore
regenerated snapshot for minor off-by-one pixel in text
3dc2011
@ericpassmore
Merge pull request #47 from eosnetworkfoundation/2025-documentation-u…
3ed74df 
…pdate

- 2025 documentation update for testing ledger. 
- regenerated snapshots for minor off by one pixel difference in text
- regenerated snapshots for flex which have better display of pagination where max pages > 10
@ericpassmore
support for core.vaulta account
f6ee9d4
@ericpassmore
Updated Tests for A Token
c983fe9
@ericpassmore
Updates buyram, buyrambytes to match production ABI, updated to Vault…
bdc87e8 
…a images
@ericpassmore
updated test screen captures with new branding
2aaa5f1
@ericpassmore
refactored tests seperate eosio from vaulta
92742be
@ericpassmore
added wumpas tests with valud and bad transfers
7ce4e44
@ericpassmore
enabled test transactions to use excoded hex data
e97e126
@ericpassmore
updated snapshots with new test coverage
79c9b1f
@ericpassmore
moved bad transaction to unknown action tests
1872429
@ericpassmore
separate test-dir for transaction refused and labeled transactions as…
00be133 
… unknown/suspect when they data is not sufficient
@ericpassmore
fix spelling; removed un-needed initalization in Action encode
be5f7c1
@ericpassmore
pylint fixes spelling and ignore multi-param warning
dd2573f
@ericpassmore
pylint updated disable too-many-arguments to latest syntax
ea4fd13
@ericpassmore
moved pylint disable into cfg file
84a9210
@ericpassmore
pylint supress too-many-branch to support more actions in tests
b86af7e
@ericpassmore
proper placement of pylint disable no members
40a89d6
@ericpassmore
linter updates
1550fcc
@ericpassmore
reverted back to Ledger Approved Variant
11cd4e5
@ericpassmore
reverted back to Ledger Approved Variant
1904a4f
@ericpassmore
reverted back to Ledger Approved App Name
6e39373
@ericpassmore
removed nanos reference; updated appName and variant
81b0444
@ericpassmore
resolved variant conflict in merge
a1eb590
@ericpassmore
Changing App Name Back to fix ending screen on Flex/Stax
54d9b9a
@ericpassmore
added validation check to print/deserialize function
678d06b
@ericpassmore ericpassmore mentioned this pull request May 16, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@ericpassmore