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

[SW-737, SW-765] AirGap Integration #1829

Closed
wants to merge 6 commits into from

Conversation

ifaouibadi
Copy link
Member

@ifaouibadi ifaouibadi commented Feb 16, 2023

v1Screen.Recording.2023-03-30.at.12.38.52.mov

@ifaouibadi ifaouibadi force-pushed the feature/mvp-qr-based-wallet branch 4 times, most recently from a1ab762 to 35dc133 Compare February 16, 2023 10:54
@ifaouibadi ifaouibadi force-pushed the feature/mvp-qr-based-wallet branch 4 times, most recently from 64eac5a to 34b0914 Compare March 6, 2023 08:19
@github-actions
Copy link

github-actions bot commented Mar 6, 2023

@ifaouibadi ifaouibadi force-pushed the feature/mvp-qr-based-wallet branch 7 times, most recently from 012b868 to 9d243c2 Compare March 13, 2023 07:00
@ifaouibadi ifaouibadi changed the title MVP(tmp): the ability to connect QR based hardware wallet [SW-737] the ability to connect QR based hardware wallet Mar 13, 2023
@ifaouibadi ifaouibadi marked this pull request as ready for review March 13, 2023 07:00
@ifaouibadi ifaouibadi force-pushed the feature/mvp-qr-based-wallet branch from 9d243c2 to f800b62 Compare March 13, 2023 09:09
@ifaouibadi ifaouibadi force-pushed the feature/mvp-qr-based-wallet branch 3 times, most recently from 4ba9d99 to 7de8de0 Compare March 16, 2023 08:21
@ifaouibadi ifaouibadi force-pushed the feature/mvp-qr-based-wallet branch from 7de8de0 to ce7c67e Compare March 22, 2023 06:00
@ifaouibadi ifaouibadi force-pushed the feature/mvp-qr-based-wallet branch from 76a65b1 to 57cbae0 Compare April 10, 2023 13:18
@ifaouibadi ifaouibadi force-pushed the feature/mvp-qr-based-wallet branch from 57cbae0 to 58d3038 Compare April 18, 2023 12:54
@ifaouibadi ifaouibadi force-pushed the feature/mvp-qr-based-wallet branch 3 times, most recently from 9f981fa to b5f132f Compare April 26, 2023 06:32
@ifaouibadi ifaouibadi force-pushed the feature/mvp-qr-based-wallet branch 4 times, most recently from 7ef505e to ba30bae Compare May 10, 2023 06:56
src/composables/airGap.ts Outdated Show resolved Hide resolved
src/composables/notifications.ts Outdated Show resolved Hide resolved
src/popup/components/AccountImportRow.vue Outdated Show resolved Hide resolved
@ifaouibadi ifaouibadi force-pushed the feature/mvp-qr-based-wallet branch 2 times, most recently from 9b36194 to 6067e9d Compare May 15, 2023 20:46
@ifaouibadi ifaouibadi requested a review from subhod-i May 15, 2023 20:47
@ifaouibadi ifaouibadi force-pushed the feature/mvp-qr-based-wallet branch 2 times, most recently from e63779e to 9f28f73 Compare June 7, 2023 10:30
@ifaouibadi ifaouibadi force-pushed the feature/mvp-qr-based-wallet branch from 9f28f73 to eea2955 Compare June 15, 2023 07:15
@davidyuk
Copy link
Member

davidyuk commented Jul 3, 2023

I'd like to integrate this on sdk side, it is still blocked by AirGap?

@ifaouibadi ifaouibadi force-pushed the feature/mvp-qr-based-wallet branch from 3fe708f to 9ca4aa0 Compare July 4, 2023 06:10
@ifaouibadi ifaouibadi force-pushed the feature/mvp-qr-based-wallet branch from 2d21a8e to 31b6774 Compare July 4, 2023 06:46
@ifaouibadi
Copy link
Member Author

I'd like to integrate this on sdk side, it is still blocked by AirGap?

@davidyuk Currently AirgGapVault deep link redirection is hard-coded to AirGapWallet.
airgap-it/airgap-vault#193

we will go with only QR code-based transaction signing for this PR

@martinkaintas
Copy link
Contributor

closing in favor of #2983

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.

6 participants