Skip to content

feat: update feature scope for Scheduler and Training Journey modules - #174

Open
umarraza086 wants to merge 1 commit into
stagingfrom
fix-scope-file
Open

feat: update feature scope for Scheduler and Training Journey modules#174
umarraza086 wants to merge 1 commit into
stagingfrom
fix-scope-file

Conversation

@umarraza086

Copy link
Copy Markdown
Collaborator

Pull Request: [Feature/Fix Description]

📝 Description

Closes: [JIRA-XXXX or GitHub Issue #XXX]

🎯 Type of Change

  • New feature
  • Bug fix
  • Documentation update
  • Performance improvement
  • Refactoring
  • Dependency update

🔍 Changes Made

  • Change 1
  • Change 2
  • Change 3

🧪 E2E Testing Checklist

Required before merging to staging:

  • Feature tested locally: npm run dev
  • No console errors (DevTools → Console)
  • Signup test: created test user testcoach+staging+[random]@example.com
  • Baseline assessment: answered questions, received persona (A/B/C/D)
  • Module viewing: clicked Module 1, video played
  • Content gate: video 90% watched before quiz unlocks
  • Quiz: answered questions, score calculated correctly
  • Module completion: passed at 80%+ score
  • Dashboard: shows completed modules and next module
  • Endline: accessible only after all modules passed
  • Certificate: generated and PDF downloads correctly
  • Supabase: new user appears in auth.users table
  • Supabase: new profiles/assessments created in correct tables
  • Mobile responsive: tested at 375px viewport
  • No sensitive data in code: no API keys, passwords, emails hardcoded
  • Code follows project conventions and standards

📸 Screenshots (if UI change)

🔄 Deployment Checklist

  • Tested on staging environment (if auto-deployed)
  • No breaking changes to database schema (or migrations included)
  • Environment variables documented (if new ones added)
  • Performance: page load time acceptable

🚨 Breaking Changes

  • This PR introduces breaking changes (describe below)
  • Requires data migration
  • Requires configuration change

If checked, describe the impact:

📋 Checklist

  • Code follows project standards (DEVELOPMENT_STANDARDS.md)
  • No hardcoded secrets or sensitive data
  • Tests added/updated (if applicable)
  • Documentation updated (if needed)
  • Commit messages are clear and descriptive
  • No merge conflicts with staging branch

🔗 Related Issues

  • Related to: [Issue/PR]
  • Depends on: [Issue/PR]
  • Blocked by: [Issue/PR]

👀 Reviewers

⚠️ Notes for Reviewers


DO NOT MERGE until:

  1. ✅ Code review approved
  2. ✅ All E2E tests pass (checkbox above)
  3. ✅ CI/CD (GitHub Actions) passes

Remember: This goes to staging first, then production after staging QA passes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants