Skip to content
This repository has been archived by the owner on Jul 1, 2021. It is now read-only.

Update Deposit and DepositData data structure #149

Closed
hwwhww opened this issue Jan 10, 2019 · 2 comments · Fixed by #181
Closed

Update Deposit and DepositData data structure #149

hwwhww opened this issue Jan 10, 2019 · 2 comments · Fixed by #181
Labels
eth2.0 good first issue Good for newcomers

Comments

@hwwhww
Copy link
Contributor

hwwhww commented Jan 10, 2019

What is wrong?

Per ethereum/consensus-specs#388, some renamings and adjustment.

How can it be fixed

  • Update eth2.beacon.types.deposits.Deposit fields
  • Update eth2.beacon.types.deposit_data.DepositData fields
@hwwhww hwwhww added good first issue Good for newcomers eth2.0 labels Jan 10, 2019
@ralexstokes
Copy link
Member

i can tackle this one @hwwhww

@hwwhww
Copy link
Contributor Author

hwwhww commented Jan 11, 2019

@ralexstokes thank you! 👍

@ralexstokes ralexstokes mentioned this issue Jan 17, 2019
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
eth2.0 good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Update deposit structures ralexstokes/trinity
2 participants
@ralexstokes @hwwhww