Skip to content

fix: Update broken miniapps links and add redirect rules #192

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

royphilip
Copy link

Summary

  • Fixed 35 broken internal links across 11 documentation files caused by path reorganization from /base-app/build-with-minikit/ to /base-app/miniapps/
  • Added redirect rules to preserve SEO and handle legacy URLs

Changes Made

  • Updated internal links in documentation files to use new miniapps paths
  • Added redirect configuration in docs.json for legacy URL handling

Impact

  • Fixes broken user experience from commit 7abc5c5
  • Preserves SEO value of existing links
  • Maintains backward compatibility for bookmarked URLs

Known Issue

Redirect rules don't work in local development (Mintlify configuration issue). New URLs work correctly, but legacy URL testing requires production environment.

- Updated all internal links from /base-app/build-with-minikit/* to /base-app/miniapps/*
- Added redirect rules in docs.json to handle legacy URLs
- Fixed 35 broken links across 11 documentation files
- Redirects should preserve SEO and user experience for existing bookmarks

Note: Redirect rules are not working in local development due to apparent
configuration issues with Mintlify in this repository. The new URLs work
correctly, and all internal links have been updated.
@cb-heimdall
Copy link
Collaborator

🟡 Heimdall Review Status

Requirement Status More Info
Reviews 🟡 0/2
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 1
Sum 2

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