Skip to content

docs: fix priority values and permalink in FAQ doc#2712

Open
kayx23 wants to merge 3 commits into
masterfrom
docs/patch-FAQ
Open

docs: fix priority values and permalink in FAQ doc#2712
kayx23 wants to merge 3 commits into
masterfrom
docs/patch-FAQ

Conversation

@kayx23
Copy link
Copy Markdown
Member

@kayx23 kayx23 commented Feb 3, 2026

Type of change:

  • Bugfix
  • New feature provided
  • Improve performance
  • Backport patches
  • Documentation
  • Refactor
  • Chore
  • CI/CD or Tests

What this PR does / why we need it:

Fix priority value and permalink in FAQ doc.

Pre-submission checklist:

  • Did you explain what problem does this PR solve? Or what new features have been added?
  • Have you added corresponding test cases?
  • Have you modified the corresponding document?
  • Is this PR backward compatible? If it is not backward compatible, please discuss on the mailing list first

@kayx23 kayx23 requested review from AlinsRan and ronething February 4, 2026 09:03
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 6, 2026

This pull request has been marked as stale due to 60 days of inactivity. It will be closed in 30 days if no further activity occurs. If you think that's incorrect or this pull request should instead be reviewed, please simply write any comment. Even if closed, you can still revive the PR at any time or discuss it on the dev@apisix.apache.org list. Thank you for your contributions.

@github-actions github-actions Bot added the stale label Apr 6, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 6, 2026

This pull request/issue has been closed due to lack of activity. If you think that is incorrect, or the pull request requires review, you can revive the PR at any time.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the English FAQ documentation around route priority behavior to correct the referenced HTTPRoute priority implementation and the numeric guidance for choosing an APISIXRoute priority value.

Changes:

  • Updates the HTTPRoute priority reference link to point at the current calculateHTTPRoutePriority implementation.
  • Adjusts the documented APISIXRoute priority value needed to outrank HTTPRoute-derived routes.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docs/en/latest/FAQ.md Outdated
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 2 comments.

Comment thread docs/en/latest/FAQ.md Outdated
Comment thread docs/en/latest/FAQ.md Outdated
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