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.

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

feat: support gasless transactions #1774

Draft
wants to merge 32 commits into
base: master
Choose a base branch
from
Draft

feat: support gasless transactions #1774

wants to merge 32 commits into from

Conversation

arthurgeron
Copy link
Contributor

@arthurgeron arthurgeron commented Jan 15, 2025 •
edited
Loading

  • Fuel SDK updated to 0.98.0, breaking changes (e.g. Provide.create, async provider calls, removal of AbstractAddress) are covered. Read here for more info.
  • Gasless transactions are now supported via the skipCustomFee param

@arthurgeron arthurgeron force-pushed the ag/fe/feat-1205 branch 3 times, most recently from cb55080 to 47bb239 Compare January 16, 2025 15:24
@github-actions GitHub Actions
Copy link
Contributor

This PR is published in NPM with version ****

@helciofranco @arthurgeron
feat: add NFTs tab (#1762)
bd3626f 
- Closes #1686 
- Closes `FE-1087`

---


| 📷 Demo |
| --- |
| <video
src="https://github.com/user-attachments/assets/d2771cb6-fe1a-4c83-9ab6-db2be421d1c6"
/> |
LuizAsFight
LuizAsFight requested changes Jan 18, 2025
View reviewed changes
Copy link
Contributor

@LuizAsFight LuizAsFight left a comment

Choose a reason for hiding this comment

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

we need to remove bech32 completely, the only screen that can have it is the Send page.

this PR needs more work to get to finish line I will work on it from now

LuizAsFight
LuizAsFight requested changes Jan 19, 2025
View reviewed changes
Copy link
Contributor

@LuizAsFight LuizAsFight left a comment

Choose a reason for hiding this comment

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

this PR should be merged only after we do the next release.

@LuizAsFight
Copy link
Contributor

needs to finish fixing e2e-contract can you take a look @arthurgeron ?

This was referenced Jan 19, 2025
Closed
Merged
@arthurgeron
Copy link
Contributor Author

needs to finish fixing e2e-contract can you take a look @arthurgeron ?

Fixed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@LuizAsFight LuizAsFight LuizAsFight requested changes

@luizstacio luizstacio Awaiting requested review from luizstacio luizstacio will be requested when the pull request is marked ready for review luizstacio is a code owner

@helciofranco helciofranco Awaiting requested review from helciofranco helciofranco will be requested when the pull request is marked ready for review helciofranco is a code owner

@rodrigobranas rodrigobranas Awaiting requested review from rodrigobranas rodrigobranas will be requested when the pull request is marked ready for review rodrigobranas is a code owner

@nelitow nelitow Awaiting requested review from nelitow nelitow will be requested when the pull request is marked ready for review nelitow is a code owner

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@arthurgeron @LuizAsFight @helciofranco