Skip to content

minikit 2.0 improvements#89

Merged
andy-t-wang merged 2 commits intomainfrom
wip-docs
Apr 1, 2026
Merged

minikit 2.0 improvements#89
andy-t-wang merged 2 commits intomainfrom
wip-docs

Conversation

@andy-t-wang
Copy link
Copy Markdown
Contributor

improve docs

@mintlify
Copy link
Copy Markdown
Contributor

mintlify bot commented Apr 1, 2026

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
world 🟢 Ready View Preview Apr 1, 2026, 5:42 AM

@andy-t-wang andy-t-wang merged commit 3945183 into main Apr 1, 2026
9 checks passed
@andy-t-wang andy-t-wang deleted the wip-docs branch April 1, 2026 05:44
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 0db4b05aca

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +38 to +40
fallback: () => {
alert("Please complete the payment in World App to proceed.");
},
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Guard pay example against fallback execution path

This example now passes a fallback callback, which means MiniKit.pay() can return executedWith: "fallback", but the snippet still treats result.data as a minikit payment payload and immediately POSTs it to /api/confirm-payment. In non-World-App contexts this can send undefined/non-payment data and make the verification step fail, so the sample should branch on result.executedWith (as done in other command docs) before using payment fields.

Useful? React with 👍 / 👎.

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