Skip to content
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

feat: frontend changes for executive education courses on B2C dashboard #181

Merged
merged 1 commit into from
Aug 10, 2023

Conversation

jajjibhai008
Copy link
Contributor

@jajjibhai008 jajjibhai008 commented Aug 4, 2023

Description
This PR covers the following changes for the B2C side.

Once an ExecEd enrollment has been made, we should

  • Hide the ‘Upgrade’ CTA - there is only one enrollment mode for Executive Education courses
  • ‘Begin Course’ and ‘Resume’ CTAs should navigate learners over to their Getsmarter dashboard.
    • We will need to ensure that the correct URL and Auth0 org_id parameter are being passed to enable SSO
  • Disable additional course options
    • Unenroll, Email Settings, Share on Facebook, and Share on Twitter should all be disabled for ExecEd course cards.

JIRA -> ENT-7188

@codecov
Copy link

codecov bot commented Aug 4, 2023

Codecov Report

Patch coverage: 100.00% and project coverage change: +0.02% 🎉

Comparison is base (f1cfe3d) 96.29% compared to head (b17a854) 96.32%.

❗ Current head b17a854 differs from pull request most recent head a2c003e. Consider uploading reports for the commit a2c003e to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #181      +/-   ##
==========================================
+ Coverage   96.29%   96.32%   +0.02%     
==========================================
  Files         182      183       +1     
  Lines        1702     1715      +13     
  Branches      300      302       +2     
==========================================
+ Hits         1639     1652      +13     
  Misses         58       58              
  Partials        5        5              
Files Changed Coverage Δ
src/data/redux/app/selectors/courseCard.js 100.00% <ø> (ø)
...components/CourseCardActions/BeginCourseButton.jsx 100.00% <100.00%> (ø)
...Card/components/CourseCardActions/ResumeButton.jsx 100.00% <100.00%> (ø)
.../CourseCard/components/CourseCardActions/index.jsx 92.85% <100.00%> (+0.54%) ⬆️
...ers/CourseCard/components/CourseCardMenu/index.jsx 100.00% <100.00%> (ø)
src/containers/CourseCard/components/hooks.js 100.00% <100.00%> (ø)
src/data/constants/course.js 100.00% <100.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@jajjibhai008 jajjibhai008 merged commit be8cb85 into master Aug 10, 2023
3 checks passed
@jajjibhai008 jajjibhai008 deleted the eahmadjaved/ENT-7188 branch August 10, 2023 07:24
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.

2 participants