Skip to content

Conversation

@hermannakos
Copy link
Collaborator

Summary

  • Remove filter that excluded unpublished courses from future enrollments
  • Unpublished future courses now appear in the "Future Enrollments" section
  • Courses display with disabled state (grayed star, no open arrow, snackbar on tap)

Test plan

  1. Log in as a student enrolled in an unpublished future course
  2. Navigate to Dashboard → Courses → All Courses
  3. Verify the "Future Enrollments" section shows the unpublished course
  4. Verify the course has a grayed-out star icon (alpha 0.4)
  5. Verify there is no open arrow on the course row
  6. Tap the course row → verify snackbar shows "Unauthorized"
  7. Tap the star icon → verify snackbar shows "Inactive courses could not be added to Dashboard."

refs: MBL-19643
affects: Student
release note: Future unpublished courses now appear in the All Courses list under Future Enrollments


  • Follow-up e2e test ticket created or not needed
  • Tested in dark mode
  • Tested in light mode
  • Test in landscape mode and/or tablet
  • A11y checked
  • Approve from product

🤖 Generated with Claude Code

Remove filter that excluded unpublished courses from future enrollments,
allowing them to appear in the Future Enrollments section with disabled
state (non-favoritable, non-openable).

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@github-actions
Copy link

github-actions bot commented Feb 2, 2026

📊 Code Coverage Report

⚠️ Student

  • PR Coverage: 43.38%
  • Master Coverage: 43.39%
  • Delta: -0.01%

✅ Teacher

  • PR Coverage: 25.51%
  • Master Coverage: 25.51%
  • Delta: +0.00%

⚠️ Pandautils

  • PR Coverage: 22.73%
  • Master Coverage: 22.73%
  • Delta: -0.00%

📈 Overall Average

  • PR Coverage: 30.54%
  • Master Coverage: 30.54%
  • Delta: -0.00%

@github-actions
Copy link

github-actions bot commented Feb 2, 2026

🧪 Unit Test Results

✅ 📱 Student App

  • Tests: 1225 total, 0 failed, 0 skipped
  • Duration: 0.000s
  • Success Rate: 100%

✅ 🌅 Horizon

  • Tests: 531 total, 0 failed, 0 skipped
  • Duration: 32.897s
  • Success Rate: 100%

✅ 📦 Submodules

  • Tests: 2706 total, 0 failed, 0 skipped
  • Duration: 51.492s
  • Success Rate: 100%

📊 Summary

  • Total Tests: 4462
  • Failed: 0
  • Skipped: 0
  • Status: ✅ All tests passed!

Last updated: Mon, 02 Feb 2026 11:26:25 GMT

@github-actions
Copy link

github-actions bot commented Feb 2, 2026

Student Install Page

Copy link
Contributor

@adamNagy56 adamNagy56 left a comment

Choose a reason for hiding this comment

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

  • I think it would be better to show the Future Enrollments section even when a user is added to a course that has never been published. Currently, it only appears if a user is added to a published course and then the course is unpublished after. On the web, the Future Enrollments section is shown correctly even when a user is added to an unpublished course.

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.

3 participants