Skip to content

UAI dashboard - unenrolled module card changes #2231

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

Merged
merged 3 commits into from
May 7, 2025

Conversation

gumaerc
Copy link
Contributor

@gumaerc gumaerc commented May 7, 2025

What are the relevant tickets?

https://github.com/mitodl/hq/issues/7255

Description (What does it do?)

This PR solves the first two items in the above issue:

  • The DashboardCard context menu is now hidden if enrollment.status is not either EnrollmentStatus.Completed or EnrollmentStatus.Enrolled
  • The new unenrolled status indicator was added

Screenshots (if appropriate):

image
image

How can this be tested?

  • Ensure that you have a Posthog project configured and have run the basic setup for mit-learn
  • Enable both the enrollment-dashboard and mitlearn-organization-dashboard feature flags
  • Visit the dashboard at http://open.odl.local:8062/dashboard (or wherever your site is hosted), logging in if necessary
  • The "My Learning" dashboard should be unchanged by this PR
  • Click either "Organization X" or Organization Y" to open the UAI dashboard
  • Ensure that on unenrolled modules, you see the new filled grey circle status indicator
  • Ensure that the context menu (3 dots) are hidden on unenrolled modules

@gumaerc gumaerc added the Needs Review An open Pull Request that is ready for review label May 7, 2025
Copy link

github-actions bot commented May 7, 2025

OpenAPI Changes

Show/hide No detectable change.

Copy link

github-actions bot commented May 7, 2025

OpenAPI Changes

Show/hide No detectable change.

@shanbady shanbady self-assigned this May 7, 2025
Copy link
Contributor

@shanbady shanbady left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@gumaerc gumaerc merged commit 3e930ee into main May 7, 2025
13 checks passed
@gumaerc gumaerc deleted the cg/enrollments-dashboard-hide-context-not-enrolled branch May 7, 2025 18:01
This was referenced May 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs Review An open Pull Request that is ready for review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants