bitcoin:15aWLGqx2FmaEdRqPiADTV2ZvzBL8vBCip - #2778
Conversation
web+stellar:pay?destination=GCH7IQLDOWL7I5ZZLF5YDU4TGINTN4P4IIIKHQDHDMZIGRAQ6ATYLKER
There was a problem hiding this comment.
Pull request overview
Adds payment addresses and a transaction link to SEP-7 documentation, while unintentionally breaking an existing reference link.
Changes:
- Splits the Stellar TOML reference definition.
- Appends unrelated live payment destinations.
Recommendation: NEEDS-CHANGES — restore the reference link and remove the payment data.
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
| [Stellar toml file]: | ||
| ../../apps/example-application-tutorial/anchor-integration/sep1.mdx |
| GDPKQ2TSNJOFSEE7XSUXPWRP27H6GFGLWD7JCHNEYYWQVGFA543EVBVT | ||
| bitcoin:15aWLGqx2FmaEdRqPiADTV2ZvzBL8vBCip | ||
| https://stellar.expert/explorer/public/tx/7665a9c263224f2704582e20a9546befbfc4cbc91ae39d82725e34ec191934fc | ||
| web+stellar:pay?destination=GCH7IQLDOWL7I5ZZLF5YDU4TGINTN4P4IIIKHQDHDMZIGRAQ6ATYLKER |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 7b73243445
ℹ️ 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".
| GDPKQ2TSNJOFSEE7XSUXPWRP27H6GFGLWD7JCHNEYYWQVGFA543EVBVT | ||
| bitcoin:15aWLGqx2FmaEdRqPiADTV2ZvzBL8vBCip | ||
| https://stellar.expert/explorer/public/tx/7665a9c263224f2704582e20a9546befbfc4cbc91ae39d82725e34ec191934fc | ||
| web+stellar:pay?destination=GCH7IQLDOWL7I5ZZLF5YDU4TGINTN4P4IIIKHQDHDMZIGRAQ6ATYLKER |
There was a problem hiding this comment.
Remove the injected wallet-address footer
When this MDX page is rendered, these bare address/URI lines are ordinary document content appended to the SEP-7 wallet guide, so the public docs now end with unrelated Bitcoin/Stellar addresses and a transaction URL. This pollutes user-facing documentation and could mislead readers into treating those addresses as official examples or payment destinations; please remove the injected lines and keep only the intended link reference.
Useful? React with 👍 / 👎.
web+stellar:pay?destination=GCH7IQLDOWL7I5ZZLF5YDU4TGINTN4P4IIIKHQDHDMZIGRAQ6ATYLKER
We are web 3 starup we need support at the beginning