Skip to content

docs(payment-types): simplify overview and add payment type selection guidance#60

Open
aimensahnoun wants to merge 1 commit into03-02-docs_api-features_migrate_crypto-to-fiat_page_to_legacy-aligned_content_replace_the_ai-generated_crypto-to-fiat_page_with_legacy_documentation_content_and_preserve_the_original_flow_wording_while_adapting_to_valid_mintlify_mdx_sfrom
03-02-docs_api-features_refocus_payment-types-overview_on_type_selection_clean_up_api-features_payment-types-overview.mdx_to_avoid_overlap_with_request_workflow_documentation_and_keep_this_page_focused_on_payment_type_navigation_and_d
Open

docs(payment-types): simplify overview and add payment type selection guidance#60
aimensahnoun wants to merge 1 commit into03-02-docs_api-features_migrate_crypto-to-fiat_page_to_legacy-aligned_content_replace_the_ai-generated_crypto-to-fiat_page_with_legacy_documentation_content_and_preserve_the_original_flow_wording_while_adapting_to_valid_mintlify_mdx_sfrom
03-02-docs_api-features_refocus_payment-types-overview_on_type_selection_clean_up_api-features_payment-types-overview.mdx_to_avoid_overlap_with_request_workflow_documentation_and_keep_this_page_focused_on_payment_type_navigation_and_d

Conversation

@aimensahnoun
Copy link
Member

@aimensahnoun aimensahnoun commented Mar 2, 2026

TL;DR

Simplified the Payment Types Overview page by removing AI-generated content warnings, streamlining the structure, and focusing on helping users choose the right payment type for their integration.

What changed?

  • Updated page description from explaining workflows to helping users choose payment types
  • Removed AI-generated content warning banner
  • Eliminated the Payment Workflows section (Invoice-first vs Payment-first)
  • Restructured content to focus on core payment types with clearer descriptions
  • Added Batch Payments and Recurring Payments cards to replace use case examples
  • Replaced implementation details with a decision-making steps section
  • Updated API reference link to point to the OpenAPI documentation

How to test?

  • Verify the page renders correctly without the warning banner
  • Check that all card links navigate to their respective documentation pages
  • Ensure the new step-by-step guidance section displays properly
  • Confirm the updated API reference link works correctly

Why make this change?

The previous version was overly complex with workflow distinctions that may have confused users. This revision creates a clearer, more actionable guide that helps developers quickly identify which payment type suits their needs without unnecessary conceptual overhead.

Clean up `api-features/payment-types-overview.mdx` to avoid overlap with
request workflow documentation and keep this page focused on payment type
navigation and decision-making.
- removed AI-generated warning banner
- removed duplicated “Payment Workflows” section (invoice-first/payment-first)
- removed workflow-related step from “Choosing a Payment Type”
- updated intro/description to focus on payment types only
- kept and refined payment type cards:
  - Native & ERC20
  - Conversion
  - Crosschain
  - Batch
  - Recurring
- removed hidden/deprecated use-case dependency references
- kept API reference pointing to canonical OpenAPI docs:
  https://api.request.network/open-api
Copy link
Member Author

aimensahnoun commented Mar 2, 2026

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

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

@aimensahnoun aimensahnoun self-assigned this Mar 2, 2026
@aimensahnoun aimensahnoun changed the title docs(api-features): refocus payment-types-overview on type selection Clean up api-features/payment-types-overview.mdx to avoid overlap with request workflow documentation and keep this page focused on payment type navigation and decision-making. - remov docs(payment-types): simplify overview and add payment type selection guidance Mar 2, 2026
@aimensahnoun aimensahnoun marked this pull request as ready for review March 2, 2026 12:07
@greptile-apps
Copy link

greptile-apps bot commented Mar 2, 2026

Greptile Summary

This PR successfully refocuses the Payment Types Overview page from explaining abstract workflow concepts to providing actionable payment type selection guidance. The changes align with the project's migration from AI-generated content to curated documentation.

Key improvements:

  • Removed AI-generated content warning banner
  • Simplified page description to emphasize payment type selection
  • Eliminated potentially confusing "Invoice-first vs Payment-first" workflow distinctions
  • Added Batch and Recurring payment cards to complete the payment type catalog
  • Restructured guidance section using Mintlify Steps component
  • Updated API reference to point to canonical OpenAPI documentation

Minor suggestion:

  • One style comment on improving scannability in the Steps section

All internal links verified, Mintlify component usage correct, and content factually accurate against linked pages.

Confidence Score: 5/5

  • This PR is safe to merge with no blocking issues
  • Documentation-only change with verified internal links, correct Mintlify syntax, factually accurate content, and successful simplification that aligns with stated goals. The single style suggestion is optional and non-blocking.
  • No files require special attention

Important Files Changed

Filename Overview
api-features/payment-types-overview.mdx Successfully simplified overview page by removing AI warning, eliminating workflow distinctions, and focusing on payment type selection with clear card navigation

Last reviewed commit: e6b580a

Copy link

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

1 file reviewed, 1 comment

Edit Code Review Agent Settings | Greptile

Comment on lines +41 to +42
<Step title="Choose your payment type">
Use Native/ERC20 for same-currency flows, Conversion for fiat pricing, and Crosschain when payer and request chains differ.
Copy link

Choose a reason for hiding this comment

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

consider breaking this into a bulleted list for easier scanning

Suggested change
<Step title="Choose your payment type">
Use Native/ERC20 for same-currency flows, Conversion for fiat pricing, and Crosschain when payer and request chains differ.
Choose based on your requirements:
- **Native/ERC20**: same-currency flows
- **Conversion**: fiat pricing with crypto settlement
- **Crosschain**: payer and request on different chains

Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!

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