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

Fixes, Examples and UI Hints #335

Merged
merged 11 commits into from
Dec 20, 2024
Merged

Fixes, Examples and UI Hints #335

merged 11 commits into from
Dec 20, 2024

Conversation

mateosauton
Copy link
Contributor

No description provided.

Copy link

vercel bot commented Dec 17, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
world-id-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 20, 2024 6:59pm

**Use Case:** This command is useful for applications that need to sign messages for verification purposes,
ensuring that the message was indeed created by the owner of the wallet.

**Example:** An app that requires users to sign a message to prove ownership
Copy link
Contributor

Choose a reason for hiding this comment

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

why would they use this in this case, instead of walletAuth?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

but its different, imagine you want to vote with your wallet you are not going to use wallet auth, you need to permform an action

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ill add the voting app example for better understanding

@@ -6,6 +6,12 @@ The verify command lets you use <Link href="/world-id/id/cloud">incognito action

To use incognito actions, first create one in the <Link href="https://developer.worldcoin.org/">Developer Portal</Link>.

**Use Case:** This command is crucial for applications that require user verification to access certain features,
ensuring that only legitimate users can perform sensitive actions.
Copy link
Contributor

Choose a reason for hiding this comment

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

I think it's also worth mentioning here that we can limit the number of times the user can perform the action - unless i'm mixing it up with something else

Copy link
Contributor Author

Choose a reason for hiding this comment

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

nope, you are right. Setting up the incognito action for the user


We recommend you avoid scroll bounce error on iOS devices. This error might happen when you use vh, vw or 100%.
Copy link
Contributor

Choose a reason for hiding this comment

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

need to adjust this to match the code below

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done


We recommend you avoid scroll bounce error on iOS devices.
Try disabling autoscroll & maybe fixed position elements or using 100dvh instead of 100vh.
Copy link
Contributor

Choose a reason for hiding this comment

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

NIT double space ...or using...

```{{ title: 'Redirect URI Example' }}
https://example.com/path/to/auth/calback/worldcoin
```
```{{ title: 'Next.js example' }}
Copy link
Contributor

Choose a reason for hiding this comment

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

with the above, this section has little sense?

@mateosauton mateosauton merged commit e062c72 into main Dec 20, 2024
9 checks passed
@mateosauton mateosauton deleted the align-rebrand branch December 20, 2024 19:00
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