Skip to content

BREAKING: Support updated SLIP44 format#25

Merged
FrederikBolding merged 3 commits into
mainfrom
fb/fix-format
Nov 11, 2022
Merged

BREAKING: Support updated SLIP44 format#25
FrederikBolding merged 3 commits into
mainfrom
fb/fix-format

Conversation

@FrederikBolding
Copy link
Copy Markdown
Member

Adds support for the slightly changed SLIP44 format and rebuild JSON.

@FrederikBolding FrederikBolding requested a review from a team as a code owner November 11, 2022 11:25
@Mrtenz
Copy link
Copy Markdown
Member

Mrtenz commented Nov 11, 2022

Should this be marked as breaking?

@FrederikBolding
Copy link
Copy Markdown
Member Author

Should this be marked as breaking?

Why?

@Mrtenz
Copy link
Copy Markdown
Member

Mrtenz commented Nov 11, 2022

It removes a property that was previously available.

@FrederikBolding
Copy link
Copy Markdown
Member Author

It removes a property that was previously available.

It was previously optional, but we can mark as breaking if need be 🤔

@Mrtenz
Copy link
Copy Markdown
Member

Mrtenz commented Nov 11, 2022

Packages could be relying on it, as it was always available for certain networks.

@FrederikBolding FrederikBolding changed the title Support updated SLIP44 format BREAKING: Support updated SLIP44 format Nov 11, 2022
@FrederikBolding
Copy link
Copy Markdown
Member Author

Packages could be relying on it, as it was always available for certain networks.

Fair 👍

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.

2 participants