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

DRAFT: @turnkey/native-passkey-stamper package #122

Closed
wants to merge 2 commits into from

Conversation

r-n-o
Copy link
Contributor

@r-n-o r-n-o commented Sep 25, 2023

Summary & Motivation

Pulling the react native stamper developed by @Mperhats in this PR; there's probably more work to do to integrate it with CI + release a minimal example using it, but I think it's useful to have this up here for inspection!

TODOs

  • make CI happy
  • ensure unit tests run as part of CI
  • ensure that this new package is released inside of codesandbox
  • write a minimal example using the new package

Co-authored-by: Michael Perhats <[email protected]>
@codesandbox-ci
Copy link

codesandbox-ci bot commented Sep 25, 2023

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'm not sure but I think these files shouldn't be checked in right? commonjs, module, typescript files are all compiled from the main ts files I believe.

@socket-security
Copy link

New dependencies detected. Learn more about Socket for GitHub ↗︎

Packages Version New capabilities Transitives Size Publisher
pod-install 0.1.0 environment +0 176 kB brentvatne
commitlint 17.0.2 None +4 64.8 MB escapedcat
react-native-builder-bob 0.20.0 filesystem, shell, environment +0 94.9 kB satya164
eslint-config-prettier 8.5.0 None +0 18.2 kB lydell
jest 28.1.1 None +3 64.8 MB simenb
react-native 0.70.3 network, filesystem, shell, environment +13 117 MB fb
release-it 15.0.0 shell, environment +5 775 kB webpro
typescript 4.5.2 None +0 64 MB typescript-bot

@r-n-o r-n-o force-pushed the rno/native-passkey-stamper branch from 9bb6722 to 4fe68d4 Compare September 25, 2023 14:37
@r-n-o
Copy link
Contributor Author

r-n-o commented Feb 1, 2024

See #209 for a new approach leveraging an existing OSS library for the tricky (native) parts! Closing this PR for now.

@r-n-o r-n-o closed this Feb 1, 2024
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