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

Feat/versioned contract #155

Merged
merged 5 commits into from
Jan 12, 2024
Merged

Feat/versioned contract #155

merged 5 commits into from
Jan 12, 2024

Conversation

neithanmo
Copy link
Collaborator

@neithanmo neithanmo commented Jan 10, 2024

This adds support for versioned smart contract. see #153

🔗 zboto Link

@ftheirs
Copy link
Contributor

ftheirs commented Jan 10, 2024

Just two minor comments that will be required by Ledger, everything else looks good to me 💪

  • Remove please linker scripts: app/script_*.ld
  • Add in the root ledger_app.toml and update guidelines_enforcer.yaml (reference)

@neithanmo neithanmo merged commit 6e45b07 into main Jan 12, 2024
34 of 37 checks passed
@neithanmo neithanmo deleted the feat/versioned_contract branch January 12, 2024 19:59
@kyranjamie
Copy link
Contributor

Hey @neithanmo. Did the ledger app not work on Stax? What's required to get Stax support?

@edgarkhanzadian
Copy link

@neithanmo can you, please, publish the package to npm with this fix?

@neithanmo
Copy link
Collaborator Author

@neithanmo can you, please, publish the package to npm with this fix?

Current published version must work without any issue, changes have been done directly to the parser layer and does not required any updates to the npm package.

@edgarkhanzadian
Copy link

@neithanmo oh right! Should it be released to ledger store then? I think the latest version there is "v0.23.3"

neithanmo added a commit that referenced this pull request Jul 24, 2024
* feat: Add support for versioned smart contracts

* bump app version and update snapshots

* remove support for stax device

* remove linker scripts

* update Ledger guidelines enforcer
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.

4 participants