Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

Update Paras module to account for candidate-descriptor / commitments split #1357

Closed
rphmeier opened this issue Jul 4, 2020 · 1 comment · Fixed by #1413
Closed

Update Paras module to account for candidate-descriptor / commitments split #1357

rphmeier opened this issue Jul 4, 2020 · 1 comment · Fixed by #1413
Assignees
Labels
S1-implement PR/Issue is in the implementation stage
Milestone

Comments

@rphmeier
Copy link
Contributor

rphmeier commented Jul 4, 2020

After #1276 - The guide section on the Paras module has diverged from the implementation ever since #1294 , where a storage optimization was made. The storage should be made to match what is described in the guide.

@rphmeier rphmeier added this to the Twinbrook milestone Jul 4, 2020
@rphmeier rphmeier added Q2-easy S1-implement PR/Issue is in the implementation stage labels Jul 4, 2020
@rphmeier
Copy link
Contributor Author

Specifically, this is about the CandidatePendingAvailability logic. Although it makes more sense to have the CandidatePendingAvailability store a CandidateDescriptor instead of CandidateReceipt, as is written in the guide. So a small update to the guide to do that would be good as well alongside changing the code to match.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
S1-implement PR/Issue is in the implementation stage
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants