Skip to content

fix: update AISIX architecture diagram link in overview documentation#12

Merged
membphis merged 2 commits intomainfrom
sy/fix-image-url
Mar 31, 2026
Merged

fix: update AISIX architecture diagram link in overview documentation#12
membphis merged 2 commits intomainfrom
sy/fix-image-url

Conversation

@LiteSun
Copy link
Copy Markdown
Contributor

@LiteSun LiteSun commented Mar 31, 2026

Summary by CodeRabbit

  • Documentation
    • Updated image source references in README and documentation to use remote-hosted assets for logos and architecture diagrams.
    • Adjusted an architecture diagram's inline styling in the introduction overview to rely on hosted sizing (removed a restrictive max-width).

@LiteSun LiteSun requested a review from membphis March 31, 2026 09:40
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 31, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 36670bce-2e13-4d75-81d9-2c44ff031748

📥 Commits

Reviewing files that changed from the base of the PR and between ad9759d and cde6747.

📒 Files selected for processing (1)
  • docs/introduction/overview.md
🚧 Files skipped from review as they are similar to previous changes (1)
  • docs/introduction/overview.md

📝 Walkthrough

Walkthrough

This PR replaces local relative image paths with fully qualified remote URLs on static.api7.ai in two documentation files: README.md and docs/introduction/overview.md. One inline style (maxWidth:'800px') was removed from the overview image tag; no other content or code changed.

Changes

Cohort / File(s) Summary
Documentation Asset URLs
README.md, docs/introduction/overview.md
Replaced relative local image paths with absolute URLs hosted on static.api7.ai. Removed maxWidth:'800px' from the image style in docs/introduction/overview.md. No other textual or structural edits.

Sequence Diagram(s)

(omitted — documentation-only changes, no new multi-component control flow)

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested reviewers

  • moonming

Poem

🐇 I hopped through docs and found a trail,

Local pictures boarded a distant mail,
On static paths they now abide,
Neat and tidy, hosted with pride,
A rabbit's cheer for images worldwide. 🎨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the main change: updating the AISIX architecture diagram link in the overview documentation, which is the primary focus of the PR.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch sy/fix-image-url

Comment @coderabbitai help to get the list of available commands and usage tips.

@membphis membphis merged commit ed48723 into main Mar 31, 2026
10 checks passed
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.

2 participants