Skip to content

chore: stop tracking video-tutorial-guide.md and test-coverage-tasks.md - #87

Merged
igorms-pro merged 5 commits into
mainfrom
85-video-ad
Jan 20, 2026
Merged

chore: stop tracking video-tutorial-guide.md and test-coverage-tasks.md#87
igorms-pro merged 5 commits into
mainfrom
85-video-ad

Conversation

@igorms-pro

Copy link
Copy Markdown
Owner

These are planning documents that should not be versioned

These are planning documents that should not be versioned
@igorms-pro igorms-pro self-assigned this Jan 19, 2026
@vercel

vercel Bot commented Jan 19, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
onelink-app Ready Ready Preview, Comment Jan 20, 2026 7:29pm
onelink-landing Ready Ready Preview, Comment Jan 20, 2026 7:29pm

@igorms-pro igorms-pro linked an issue Jan 19, 2026 that may be closed by this pull request
- Remove self-redirect loop in App.tsx when on landing domain
- Only redirect app routes to app domain, not root or profiles
- Add explicit HTTP to HTTPS redirects in both vercel.json files
- Remove unused landingRoutes variable
- Prevents infinite loops when web app is served on landing domain
- Update tests to reflect that web app should not redirect when on landing domain
- Landing domain should serve landing project, not web app
- Tests now verify no redirects occur (preventing infinite loops)
@codecov

codecov Bot commented Jan 20, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

- Remove VIDEO_TUTORIAL_PLAN.md (superseded by video-tutorial-guide.md)
- Remove VIDEO_PROMOTIONAL_LANDING.md (superseded by video-tutorial-guide.md)
- Keep video-tutorial-guide.md as the comprehensive guide
@igorms-pro igorms-pro removed a link to an issue Jan 20, 2026
@igorms-pro
igorms-pro merged commit 2e825c1 into main Jan 20, 2026
13 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.

1 participant