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

Escrow: move assets from storage to data_structures::EscrowInfo #646

Open
K1-R1 opened this issue May 2, 2023 · 1 comment
Open

Escrow: move assets from storage to data_structures::EscrowInfo #646

K1-R1 opened this issue May 2, 2023 · 1 comment
Labels
App: Escrow Label used to filter for the app issue Good First Issue Good for newcomers Improvement Enhancing a feature that already exists

Comments

@K1-R1
Copy link
Member

K1-R1 commented May 2, 2023

Within the storage block in escrow-contract/src/main.sw a previously blocked TODO is now unblocked. The assets storage variable can now be moved into EscrowInfo within escrow-contract/src/data_structures.sw and any associated refactoring can now be completed.
@K1-R1 K1-R1 added Good First Issue Good for newcomers Improvement Enhancing a feature that already exists App: Escrow Label used to filter for the app issue labels May 2, 2023
@Braqzen
Copy link
Contributor

Braqzen commented May 2, 2023

I think we may need to bump before this can be implemented otherwise bumping and refactoring would be mixed into 1 PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
App: Escrow Label used to filter for the app issue Good First Issue Good for newcomers Improvement Enhancing a feature that already exists
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@K1-R1 @Braqzen