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

feat: implement new wallet trait for askar #1085

Merged
merged 5 commits into from
Feb 26, 2024
Merged

feat: implement new wallet trait for askar #1085

merged 5 commits into from
Feb 26, 2024

Conversation

xprazak2
Copy link
Contributor

@xprazak2 xprazak2 commented Dec 6, 2023

These changes include implementation of BaseWallet for askar.

These changes intentionally do not include any data migrations between wallets using BaseWallet or compatibility tests between different wallet implementations. Even though both of them are important, they are not needed for a new wallet implementation to work on its own, therefore they can (and will) be added separately as this is already large enough.

@xprazak2 xprazak2 force-pushed the indy-record-wallet branch 7 times, most recently from c572e24 to 550ae5d Compare December 21, 2023 11:10
@xprazak2 xprazak2 force-pushed the indy-record-wallet branch 10 times, most recently from d7199da to 748ab8a Compare January 3, 2024 06:58
@xprazak2 xprazak2 force-pushed the indy-record-wallet branch 5 times, most recently from bc3a5c6 to 91c0004 Compare January 4, 2024 15:16
@xprazak2 xprazak2 force-pushed the indy-record-wallet branch 3 times, most recently from f85aedc to 55cbf0a Compare January 12, 2024 08:51
Base automatically changed from indy-record-wallet to main January 12, 2024 10:48
@xprazak2 xprazak2 force-pushed the ai branch 2 times, most recently from 3fc31ec to 255acfc Compare February 19, 2024 14:45
Patrik-Stas
Patrik-Stas previously approved these changes Feb 21, 2024
Copy link
Contributor

@Patrik-Stas Patrik-Stas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, well done! ✅ Let's get this baby merged. Just need to sort out that napi image

mirgee
mirgee previously approved these changes Feb 22, 2024
Ondrej Prazak added 3 commits February 23, 2024 11:20
Signed-off-by: Ondrej Prazak <[email protected]>
Signed-off-by: Ondrej Prazak <[email protected]>
@xprazak2 xprazak2 dismissed stale reviews from mirgee and Patrik-Stas via ff2676d February 23, 2024 10:24
@xprazak2 xprazak2 force-pushed the ai branch 9 times, most recently from d8d5345 to 76876a7 Compare February 23, 2024 13:25
@xprazak2 xprazak2 force-pushed the ai branch 6 times, most recently from a63dca0 to a7db7f5 Compare February 26, 2024 06:59
Signed-off-by: Ondrej Prazak <[email protected]>
@Patrik-Stas Patrik-Stas merged commit 507428f into main Feb 26, 2024
32 checks passed
@Patrik-Stas Patrik-Stas deleted the ai branch February 26, 2024 10:06
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.

5 participants