Skip to content

Conversation

@AleemAlam
Copy link

  • Upgraded polkasafe to version 1.5.0 and added react and react-dom dependencies.
  • Introduced new getMultisigTransaction method in IPolkasafeRepository to retrieve multisig transactions.
  • Updated PolkasafeRepository to implement the new method, improving transaction handling.
  • Refactored PolkadotWalletService to utilize the new getMultisigTransaction method, enhancing the wallet's transaction capabilities.
  • Adjusted various types and parameters for better clarity and functionality.

Pull Request Summary

Describe what changes this pull request makes to the repository

Check list

  • contains breaking changes
  • adds new feature
  • modifies existing feature (bug fix or improvements)
  • relies on other tasks
  • documentation changes
  • tested on mobile devices

This pull request makes the following changes:

Adds

  • (ex: Add feature A)

Fixes

  • (ex: Fix validation function)

Changes

  • (ex: Change document B)

To-dos

*Feel free to remove this section if it's not applicable

  • (ex: add user list)

- Upgraded `polkasafe` to version 1.5.0 and added `react` and `react-dom` dependencies.
- Introduced new `getMultisigTransaction` method in `IPolkasafeRepository` to retrieve multisig transactions.
- Updated `PolkasafeRepository` to implement the new method, improving transaction handling.
- Refactored `PolkadotWalletService` to utilize the new `getMultisigTransaction` method, enhancing the wallet's transaction capabilities.
- Adjusted various types and parameters for better clarity and functionality.
- Introduced ProxyAccount interface to better manage proxy accounts in multisig.
- Updated MultisigAddress to support an array of proxy accounts and added a new multisigAddress field.
- Refactored addProxyAccounts function to handle multiple proxy accounts and sort signatories.
- Modified PolkadotWalletService to utilize the new multisigAddress for proxy accounts, improving address handling in transactions.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant