Skip to content

docs(payment-detection): simplify content and remove AI warning - #64

Merged
rodrigopavezi merged 1 commit into
mainfrom
03-02-docs_api-features_refine_payment-detection_with_legacy-aligned_accurate_guidance_update_api-features_payment-detection.mdx_to_remove_ai-generated_placeholder_framing_and_align_the_page_with_legacy-supported_detection_concepts_wh
May 12, 2026
Merged

docs(payment-detection): simplify content and remove AI warning#64
rodrigopavezi merged 1 commit into
mainfrom
03-02-docs_api-features_refine_payment-detection_with_legacy-aligned_accurate_guidance_update_api-features_payment-detection.mdx_to_remove_ai-generated_placeholder_framing_and_align_the_page_with_legacy-supported_detection_concepts_wh

Conversation

@aimen74

@aimen74aimen74 commented Mar 2, 2026

Copy link
Copy Markdown
Contributor

TL;DR

Simplified and streamlined the Payment Detection documentation by removing AI-generated content warnings and reducing verbose explanations.

What changed?

  • Removed the AI-generated content warning banner
  • Condensed the overview section to focus on core functionality
  • Simplified the "How It Works" section descriptions with more concise language
  • Replaced specific network lists with a reference to the supported chains documentation
  • Removed the detailed "Payment States" section
  • Streamlined feature cards to use shorter, more direct descriptions
  • Updated API query examples to be more specific

How to test?

Review the updated documentation page to ensure:

  • All technical information remains accurate
  • The content flows logically without the removed sections
  • Links to the supported chains documentation work correctly
  • The simplified explanations are still clear and informative

Why make this change?

The documentation was overly verbose and contained redundant information. This change makes the content more accessible and easier to digest while maintaining all essential technical details. Removing the AI-generated warning also presents the content with more confidence.

@aimen74Graphite App

aimen74 commented Mar 2, 2026

Copy link
Copy Markdown
ContributorAuthor

This stack of pull requests is managed by Graphite. Learn more about stacking.

@aimen74aimen74 self-assigned this Mar 2, 2026
@aimen74aimen74 changed the title docs(api-features): refine payment-detection with legacy-aligned, accurate guidance Update api-features/payment-detection.mdx to remove AI-generated placeholder framing and align the page with legacy-supported detection concepts while keeping Mintlify sdocs(payment-detection): simplify content and remove AI warningMar 2, 2026
@aimen74
aimen74 marked this pull request as ready for review March 2, 2026 12:10
@greptile-apps

greptile-appsBot commented Mar 2, 2026

Copy link
Copy Markdown

Greptile Summary

This PR simplifies the Payment Detection documentation by removing an AI-generated content warning, condensing prose descriptions, replacing the inline network list with a link to the supported chains page, and deleting the Payment States section.

  • The AI warning banner, verbose network lists, and "Payment States" enum table are removed; prose throughout is tightened.
  • A specific API query path (GET /v2/request/{requestId}) replaces a generic description, and the link to /resources/supported-chains-and-currencies resolves correctly.
  • The deleted Payment States section contained the actual string enum values (created, pending, partially_paid, paid, overpaid, refunded) that developers match against API responses; their removal is the main concern with this change.

Confidence Score: 4/5

Safe to merge after deciding whether to restore the Payment States enum — all other changes are straightforward prose simplifications.

The only changed file is documentation. The prose simplifications and link update are clean. The one real concern is that the Payment States section — which listed the exact state strings the API returns — was deleted without a replacement. Developers building integrations from this page won't find created, pending, overpaid, or refunded anywhere in the doc.

api-features/payment-detection.mdx — specifically around the removed Payment States content near the end of the file.

Important Files Changed

FilenameOverview
api-features/payment-detection.mdxSimplified payment detection docs by removing the AI warning, condensing prose, and replacing the network list with a link — but the Payment States section (which documented the actual API state enum values) was deleted, leaving developers without the complete state contract.

Sequence Diagram

sequenceDiagram
participant Dev as Developer
participant API as Request Network API
participant Chain as Blockchain
participant Subgraph as Subgraph Indexer
participant WH as Webhook Endpoint
Dev->>API: POST /v2/request (create request)
API-->>Dev: requestId + paymentReference (16-char)
Dev->>Chain: Payer sends tx with paymentReference
Chain-->>Subgraph: New transaction event
Subgraph->>API: Payment reference matched
API->>API: Validate amount, currency, recipient
API->>API: Update request status (created → paid)
API-->>WH: POST webhook (payment event)
Dev->>API: "GET /v2/request/{requestId}"
API-->>Dev: Updated payment status
Loading

Reviews (2): Last reviewed commit: "docs(api-features): refine payment-detec..." | Re-trigger Greptile

@greptile-appsgreptile-appsBot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

1 file reviewed, no comments

Edit Code Review Agent Settings | Greptile

@rodrigopaveziGraphite App

rodrigopavezi commented May 12, 2026

Copy link
Copy Markdown
Contributor

Merge activity

  • May 12, 4:02 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • May 12, 4:28 PM UTC: Graphite rebased this pull request as part of a merge.
  • May 12, 4:29 PM UTC: @rodrigopavezi merged this pull request with Graphite.

@rodrigopavezi
rodrigopavezi changed the base branch from 03-02-docs_api-features_rewrite_partial-payments_with_endpoint-driven_guidance_refactor_api-features_partial-payments.mdx_from_ai-generated_placeholder_content_into_accurate_implementation-oriented_documentation_aligned_with_current_r to graphite-base/64May 12, 2026 16:25
@rodrigopavezi
rodrigopavezi changed the base branch from graphite-base/64 to mainMay 12, 2026 16:27
…urate guidance
Update `api-features/payment-detection.mdx` to remove AI-generated placeholder framing and align the page with legacy-supported detection concepts while keeping Mintlify structure.
- removed AI warning banner and legacy “stable docs” dependency messaging
- tightened overview and flow language around reference-based payment detection
- kept core detection lifecycle intact:
- payment reference generation
- blockchain monitoring
- automatic validation + status updates
- webhook triggering
- clarified API access point for status checks with `GET /v2/request/{requestId}`
- replaced hardcoded supported-network lists with canonical internal reference:
`/resources/supported-chains-and-currencies`
- kept crosschain detection explanation and webhook sections, but removed over-specific/unverified claims
- removed rigid payment-state matrix to avoid inaccurate state assumptions across flows
- verified internal links and removed legacy-domain references from the page
@rodrigopavezi
rodrigopaveziforce-pushed the 03-02-docs_api-features_refine_payment-detection_with_legacy-aligned_accurate_guidance_update_api-features_payment-detection.mdx_to_remove_ai-generated_placeholder_framing_and_align_the_page_with_legacy-supported_detection_concepts_wh branch from b89f3dd to db11d4bCompareMay 12, 2026 16:28
@rodrigopavezi
rodrigopavezi merged commit 85b92f2 into mainMay 12, 2026
2 checks passed
@rodrigopavezi
rodrigopavezi deleted the 03-02-docs_api-features_refine_payment-detection_with_legacy-aligned_accurate_guidance_update_api-features_payment-detection.mdx_to_remove_ai-generated_placeholder_framing_and_align_the_page_with_legacy-supported_detection_concepts_wh branch May 12, 2026 16:29
- **`overpaid`** - Payment exceeds expected amount
- **`refunded`** - Payment was refunded

## What's Next?

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1Removal of Payment States section loses critical API contract information

The deleted "Payment States" section documented the exact state values returned by the API (created, pending, partially_paid, paid, overpaid, refunded). These are string enum values that developers must match in their code to handle webhook payloads and polling responses correctly. The replacement in section 3 — "for example partially paid or fully paid" — is informal and omits created, pending, overpaid, and refunded, meaning any developer building state-based UI or business logic from this page alone will be missing states their integration must handle.

Sign up for freeto 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.

Documentation - Merge Mintlify docs and replace legacy docs

2 participants

@aimen74@rodrigopavezi