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

Full test coverage for the basic section of the Multisig functionality #104

Closed
wants to merge 21 commits into from

Conversation

Braqzen
Copy link
Contributor

@Braqzen Braqzen commented Jun 22, 2022

Type of change

  • New feature

Changes

The following changes have been made:

  • TODO

Notes

  • Awaiting Vec support in SDK in order to test

Related Issues

Closes #7

@Braqzen Braqzen added New Feature New addition that does not currently exist Good First Issue Good for newcomers Awaiting Sway feature(s) Language does not support desired functionality, yet labels Jun 22, 2022
@Braqzen Braqzen self-assigned this Jun 22, 2022
@Braqzen Braqzen changed the title Full test coverage for Basic functionality Full test coverage for the basic section of the Multisig functionality Jun 22, 2022
@Braqzen Braqzen added the App: Multisig Label used to filter for the app issue label Jun 24, 2022
@simonr0204 simonr0204 added the Awaiting SDK feature(s) SDK does not support desired functionality, yet label Aug 23, 2022
@K1-R1
Copy link
Contributor

K1-R1 commented Dec 6, 2022

This PR can be closed once #272 is merged, as it includes the addition of full test coverage for the Multisig-wallet.

@simonr0204 simonr0204 removed Awaiting Sway feature(s) Language does not support desired functionality, yet Awaiting SDK feature(s) SDK does not support desired functionality, yet labels Dec 9, 2022
@simonr0204
Copy link
Contributor

If the other PR includes the changes, then this one will never be merged, right? We should close it, if that's the case.

@Braqzen
Copy link
Contributor Author

Braqzen commented Dec 9, 2022

If the other PR includes the changes, then this one will never be merged, right? We should close it, if that's the case.

Correct

@simonr0204 simonr0204 closed this Dec 9, 2022
@Braqzen Braqzen deleted the braqzen-7 branch February 23, 2023 18:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
App: Multisig Label used to filter for the app issue Good First Issue Good for newcomers New Feature New addition that does not currently exist
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Full test coverage
5 participants