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

Improve Address DX #3598

Open
arboleya opened this issue Jan 20, 2025 · 0 comments · May be fixed by #3600
Open

Improve Address DX #3598

arboleya opened this issue Jan 20, 2025 · 0 comments · May be fixed by #3600
Assignees
@petertonysmith94
Labels
chore Issue is a chore

Comments

@arboleya
Copy link
Member

We should consider making the Address.constructor() accept the same input as Address.fromDynamicInput().

// before
constructor(address: B256Address) {}


// after
constructor(address: string | Address) {}

This could make DX a bit nicer:

// before
Address.fromDynamicInput(xyz)

// after
new Address(xyz)
@arboleya arboleya added the chore Issue is a chore label Jan 20, 2025
@petertonysmith94 petertonysmith94 self-assigned this Jan 20, 2025
@petertonysmith94 petertonysmith94 linked a pull request Jan 20, 2025 that will close this issue
Open
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@petertonysmith94 petertonysmith94

Labels
chore Issue is a chore
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

chore: Address constructor now accepts a range of inputs. FuelLabs/fuels-ts
2 participants
@arboleya @petertonysmith94