mitodl / micromasters

Portal for learners and course teams to access MITx Micromasters® programs
https://mm.mit.edu
BSD 3-Clause "New" or "Revised" License
28 stars 16 forks source link

Remove edx logo from certificates #5325

Closed annagav closed 1 year ago

annagav commented 1 year ago

Pre-Flight checklist

What are the relevant tickets?

Fix https://github.com/mitodl/micromasters/issues/5324

What's this PR do?

Removes edx logo for a certificate. Removes explanations of edx platform and verified certificate.

How should this be manually tested?

Go to view a course certificate and a program certificate.

Before:

Screen Shot 2023-04-10 at 1 46 45 PM

After:

Screen Shot 2023-04-10 at 1 49 35 PM
codecov-commenter commented 1 year ago

Codecov Report

Merging #5325 (ef75087) into master (f33edf1) will increase coverage by 0.00%. The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@           Coverage Diff           @@
##           master    #5325   +/-   ##
=======================================
  Coverage   93.59%   93.60%           
=======================================
  Files         499      499           
  Lines       22979    22979           
  Branches      968      968           
=======================================
+ Hits        21507    21509    +2     
+ Misses       1364     1363    -1     
+ Partials      108      107    -1     

see 1 file with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

pdpinch commented 1 year ago

Thanks for asking @arslanashraf7. I've discussed this with the MITx team and, in this case, we don't need to preserve the historical logos.