Skip to content

feat(deen): fold /review checklist into the Deen journey (Review tab) - #3

Merged
mohamed-dev2 merged 1 commit into
developfrom
feat/deen-review-merge
Sep 23, 2026
Merged

mohamed-dev2 merged 1 commit into
developfrom
feat/deen-review-merge

Conversation

@mohamed-dev2

Copy link
Copy Markdown
Owner

Merges the ا'بلاد /review night checklist into /deen as a third tab (Today · Library · Review); /review now permanently redirects (308) so old bookmarks, shared links, and PWA notification deep-links keep working.

  • nav: review entry + href + nav.review strings dropped; deen becomes the single entry point
  • route: app/review/ deleted; logic lifted to app/components/views/review-tab.tsx (git-rename 92%)
  • /deen: third tab with dn.review label (المراجعة / Review, AR+EN parity 1307 keys); metadata description updated
  • SEO: sitemap, internal-footer links, title/description route guards, e2e route table updated; new e2e asserts /review → 308 → /deen
  • offline/PWA: sw.js CORE drops /review (CACHE bumped v6→v7), notify + manifest shortcuts → /deen, guide action 'review' pushes /deen
  • docs/audit: DOCUMENTATION, README, CHANGELOG, features (deen, goals-habits-review), SEO/PERFORMANCE/INDEXING/offline/ADR, AGENTS, audit/*.json regenerated (routes 11→10)

Local gates all green: typecheck, lint, unit 287, e2e 55 (incl. new redirect test), build 15 static routes, docs/comments/css/boundaries/format, npm audit 0.

Signed-off-by: mohamed-dev2 <mohammedmagdygouda@gmail.com>
@vercel

vercel Bot commented Sep 23, 2026 •

Copy link
Copy Markdown

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

Project Deployment Actions Updated
wird Ready Ready Preview Sep 23, 2026 1:30am UTC

@mohamed-dev2
mohamed-dev2 merged commit 5f38dc2 into develop Sep 23, 2026
3 checks passed
@mohamed-dev2
mohamed-dev2 deleted the feat/deen-review-merge branch September 23, 2026 01:33

This branch was successfully deployed

1 active deployment
Preview — a8021243 Deployed Sep 23, 2026 by vercel[bot]
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