Skip to content

docs(lifecycle): replace external image with local asset and remove AI-generated warning#76

Open
aimensahnoun wants to merge 1 commit into03-03-docs_resources_clean_easyinvoice_and_request_scan_pages_for_tools_examples_refine_the_tools_examples_docs_by_removing_low-value_placeholder_sections_while_keeping_the_key_product_walkthrough_content_intact._-_request-network-apifrom
03-03-docs_resources_align_lifecycle-of-a-request_with_legacy_content_and_local_assets_update_resources_lifecycle-of-a-request.mdx_to_match_legacy_lifecycle_documentation_while_removing_legacy-hosted_dependencies._-_removed_ai-generat
Open

docs(lifecycle): replace external image with local asset and remove AI-generated warning#76
aimensahnoun wants to merge 1 commit into03-03-docs_resources_clean_easyinvoice_and_request_scan_pages_for_tools_examples_refine_the_tools_examples_docs_by_removing_low-value_placeholder_sections_while_keeping_the_key_product_walkthrough_content_intact._-_request-network-apifrom
03-03-docs_resources_align_lifecycle-of-a-request_with_legacy_content_and_local_assets_update_resources_lifecycle-of-a-request.mdx_to_match_legacy_lifecycle_documentation_while_removing_legacy-hosted_dependencies._-_removed_ai-generat

Conversation

@aimensahnoun
Copy link
Member

@aimensahnoun aimensahnoun commented Mar 3, 2026

TL;DR

Replaced external GitBook image with local WebP image and updated documentation link reference.

What changed?

  • Added a new local WebP image file for the request lifecycle diagram at /images/architecture/lifecycle-of-a-request/request-lifecycle.webp
  • Replaced the external GitBook image URL with the local image reference in the Frame component
  • Removed the AI-generated content warning banner
  • Updated the documentation link from the Quickstart guide to the SDK (Legacy) overview page

How to test?

  1. Navigate to the lifecycle-of-a-request page
  2. Verify the request lifecycle diagram displays correctly
  3. Confirm the image loads from the local path instead of external GitBook URL
  4. Check that the SDK (Legacy) link at the bottom of the page works properly

Why make this change?

This change improves documentation reliability by hosting the lifecycle diagram locally instead of depending on external GitBook URLs, which could break or change. It also removes outdated AI-generated content warnings and updates navigation to point to the correct SDK documentation section.

… local assets

Update `resources/lifecycle-of-a-request.mdx` to match legacy lifecycle
documentation while removing legacy-hosted dependencies.
- removed AI-generated warning banner
- replaced external GitBook image URL with local repo asset:
  `/images/architecture/lifecycle-of-a-request/request-lifecycle.webp`
- preserved legacy lifecycle flow and section structure:
  - Create a request
  - Update a request
  - Pay a request
  - Retrieve a request / Detect a payment
- kept the Tron mention in the “Create a request” info note as requested
- replaced legacy external quickstart URL with internal docs link:
  `/sdk-legacy/overview`
- retained payment-reference and declarative/request-balance explanations
This was referenced Mar 3, 2026
Copy link
Member Author

aimensahnoun commented Mar 3, 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 changed the title docs(resources): align lifecycle-of-a-request with legacy content and local assets Update resources/lifecycle-of-a-request.mdx to match legacy lifecycle documentation while removing legacy-hosted dependencies. - removed AI-generated warning banner - rep docs(lifecycle): replace external image with local asset and remove AI-generated warning Mar 3, 2026
@aimensahnoun aimensahnoun self-assigned this Mar 3, 2026
@aimensahnoun aimensahnoun marked this pull request as ready for review March 3, 2026 14:46
@greptile-apps
Copy link

greptile-apps bot commented Mar 3, 2026

Greptile Summary

This PR improves the reliability of the lifecycle-of-a-request documentation page by replacing an external GitBook image URL with a locally hosted WebP asset and updating the trailing SDK link from an external URL to an internal Mintlify page reference.

Key changes:

  • Added images/architecture/lifecycle-of-a-request/request-lifecycle.webp as a local asset to eliminate the dependency on the external GitBook CDN URL, which could break or become unavailable.
  • Updated resources/lifecycle-of-a-request.mdx to reference the local image via <img src="/images/architecture/lifecycle-of-a-request/request-lifecycle.webp" />, consistent with the /images/ path convention used throughout the rest of the docs.
  • Removed the <Warning> AI-generated content banner, which is appropriate as the page content is being actively maintained.
  • Replaced the external https://docs.request.network/... Quickstart link with the internal /sdk-legacy/overview route, which resolves to an existing page within the Mintlify docs.
  • Fixed a missing trailing newline at the end of the MDX file.

Confidence Score: 5/5

  • This PR is safe to merge — changes are purely documentation improvements with no logic or security implications.
  • The PR makes three straightforward documentation-only changes: (1) the local image file exists at the path referenced in the MDX, (2) the /images/ path convention matches what every other page in the docs uses, (3) the internal link /sdk-legacy/overview resolves to an existing sdk-legacy/overview.mdx file. No broken links, no missing assets, no logic errors.
  • No files require special attention.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A["lifecycle-of-a-request.mdx"] -->|"references"| B["/images/architecture/lifecycle-of-a-request/request-lifecycle.webp\n(local WebP asset)"]
    A -->|"previously referenced"| C["External GitBook URL\n(removed)"]
    A -->|"links to"| D["/sdk-legacy/overview\n(internal Mintlify page)"]
    A -->|"previously linked to"| E["External Quickstart URL\n(removed)"]
    style C fill:#ffcccc,stroke:#cc0000
    style E fill:#ffcccc,stroke:#cc0000
    style B fill:#ccffcc,stroke:#007700
    style D fill:#ccffcc,stroke:#007700
Loading

Last reviewed commit: 6f231b0

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.

2 files reviewed, no comments

Edit Code Review Agent Settings | Greptile

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