Skip to content

fix: remove duplicate admin-settings route - #822

Merged
khushalsonawat merged 1 commit into
future-agi:devfrom
kimngoc280105:fix/admin-settings-duplicate-route
Aug 18, 2026
Merged

fix: remove duplicate admin-settings route#822
khushalsonawat merged 1 commit into
future-agi:devfrom
kimngoc280105:fix/admin-settings-duplicate-route

Conversation

@kimngoc280105

Copy link
Copy Markdown
Contributor

Remove src/pages/docs/admin-settings.mdx (stale page) since
src/pages/docs/admin-settings/index.mdx already covers this route
with more complete content. Astro was warning about two static
routes both resolving to /docs/admin-settings.

Testing

Ran pnpm dev locally before and after — warning is gone, and
/docs/admin-settings still renders the expected content.

Remove src/pages/docs/admin-settings.mdx (stale page) since
src/pages/docs/admin-settings/index.mdx already covers this route
with more complete content. Astro was warning about two static
routes both resolving to /docs/admin-settings.

@khushalsonawat khushalsonawat left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Hello @kimngoc280105,
Thank you for contributing to Future AGI docs. Your change looks accurate to me. I'm approving this PR and merging it
Looking forward to your future contributions

@khushalsonawat
khushalsonawat merged commit d46503f into future-agi:dev Aug 18, 2026
1 of 4 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