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

[ADP-3492] Retrieve genesis data from disk for the deposit wallet #4854

Merged
merged 6 commits into from
Nov 25, 2024

Conversation

paolino
Copy link
Collaborator

@paolino paolino commented Nov 21, 2024

This PR substitute a mock implementation of GenesisData with one loaded from the disk. Because we do not have persistence yet this is not creating conflicts at the moment.
This allow us to run the deposit wallet in preprod by supplying the preprod byron genesis file
We also opportunistically fix the UI code to encode correctly addresses so that testnet addresses are now shown correctly.

  • Load GenesisData from disk by specifying a filepath in the CLI arguments
  • Use the address encoding lib in the UI to actually show testnet addresses
  • Fix GenesisData not propagated correctly in the REST interface

ADP-3492

@paolino paolino self-assigned this Nov 21, 2024
@paolino paolino force-pushed the paolino/ADP-3492/retrieve-genesis-data-from-disk branch 3 times, most recently from 077407f to 5dfdcd6 Compare November 21, 2024 15:14
@paolino paolino marked this pull request as ready for review November 21, 2024 17:52
@paolino paolino force-pushed the paolino/ADP-3492/retrieve-genesis-data-from-disk branch from 5dfdcd6 to b5252c7 Compare November 22, 2024 08:37
Copy link
Contributor

@HeinrichApfelmus HeinrichApfelmus left a comment

Choose a reason for hiding this comment

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

Excellent, thank you! 😊 One question about debugging statement, but happy to merge in any case.

@paolino paolino added this pull request to the merge queue Nov 25, 2024
Merged via the queue into master with commit f1f8390 Nov 25, 2024
24 checks passed
@paolino paolino deleted the paolino/ADP-3492/retrieve-genesis-data-from-disk branch November 25, 2024 13:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants