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

Stackblitz environment + instructions added to README's for solana examples #395

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

omkarshanbhag
Copy link
Contributor

Summary & Motivation

How I Tested These Changes

Did you add a changeset?

If updating one of our packages, you'll likely need to add a changeset to your PR. To do so, run pnpm changeset. pnpm changeset will generate a file where you should write a human friendly message about the changes. Note how this (example) includes the package name (should be auto added by the command) along with the type of semver change (major.minor.patch) (which you should set).

These changes will be used at release time to determine what packages to publish and how to bump their version. For more context see this comment.

Copy link

codesandbox-ci bot commented Oct 16, 2024

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
Collaborator

@andrewkmin andrewkmin left a comment

Choose a reason for hiding this comment

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

comments on examples/email-auth-local-storage/README.md apply to the rest of the examples as well. let's make sure we update the READMEs and .env.local files within the individual stackblitzes.

also I recall there may have been inconsistencies with env var naming -- are there any changes to upstream back into this tkhq/sdk repo?

overall 👏👏👏

examples/email-auth-local-storage/README.md Outdated Show resolved Hide resolved

#### Prerequisites

To use the example you need the following
Copy link
Collaborator

Choose a reason for hiding this comment

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

Let's make sure to include these instructions in the stackblitz repo itself

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sounds good - I'll keep them here too cause I suspects that folks will still want to use the readme from Github UI than figuring out how to render the one in stackblitz or following along the unrendered markdown file

examples/email-auth-local-storage/README.md Show resolved Hide resolved
examples/email-auth-local-storage/README.md Show resolved Hide resolved
examples/with-eth-passkeys-galore/README.md Outdated Show resolved Hide resolved
@@ -10,6 +10,8 @@ The overall flow for email auth is outlined below:

This example contains an auth page as well and enables you to create wallets using an email auth credential. Unlike our other [email auth example](https://github.com/tkhq/sdk/tree/main/examples/email-auth), this does not use an iframe, which typically houses a target embedded key. Instead, a target embedded key is created within the application itself and stored in local storage.

You can try this example quickly on Stackblitz -- follow the instructions below
Copy link
Collaborator

Choose a reason for hiding this comment

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

we can suggest jumping to Step 4 here (and we should be able to add a link to it via markdown)

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.

2 participants