feat: replace MITx Online logo with MIT Learn logo in all emails by Anas12091101 · Pull Request #3400 · mitodl/mitxonline

Anas12091101 · 2026-03-19T11:57:12Z

What are the relevant tickets?

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

Description (What does it do?)

This PR replaces the MITxOnline Logo with the MIT Learn logo in all emails

Screenshots (if appropriate):

Unenroll

Enroll

Receipt

Refund

Flexible Price

Verify your email

Change email

How can this be tested?

Verify that all the emails sent from the MITxOnline application have the MIT Learn logo

Additional Context

github-actions · 2026-03-19T11:57:29Z

OpenAPI Changes

Show/hide ## Changes for v0.yaml:

## Changes for v0.yaml:
11 changes: 11 error, 0 warning, 0 info
error	[response-required-property-removed] at head/openapi/specs/v0.yaml	
	in API GET /api/v2/pages/?fields=*&type=cms.programpage
		removed the required property 'items/items/program_details/programs' from the response with the '200' status

error	[response-required-property-removed] at head/openapi/specs/v0.yaml	
	in API GET /api/v2/pages/{id}/
		removed the required property '/oneOf[#/components/schemas/ProgramPageItem]/program_details/programs' from the response with the '200' status

error	[response-required-property-removed] at head/openapi/specs/v0.yaml	
	in API GET /api/v2/program_certificates/{cert_uuid}/
		removed the required property 'program/programs' from the response with the '200' status

error	[response-required-property-removed] at head/openapi/specs/v0.yaml	
	in API GET /api/v2/program_enrollments/
		removed the required property '/items/program/programs' from the response with the '200' status

error	[response-required-property-removed] at head/openapi/specs/v0.yaml	
	in API DELETE /api/v2/program_enrollments/{id}/
		removed the required property '/items/program/programs' from the response with the '200' status

error	[response-required-property-removed] at head/openapi/specs/v0.yaml	
	in API GET /api/v2/program_enrollments/{id}/
		removed the required property 'program/programs' from the response with the '200' status

error	[response-required-property-removed] at head/openapi/specs/v0.yaml	
	in API GET /api/v2/programs/
		removed the required property 'results/items/programs' from the response with the '200' status

error	[response-required-property-removed] at head/openapi/specs/v0.yaml	
	in API GET /api/v2/programs/{id}/
		removed the required property 'programs' from the response with the '200' status

error	[response-required-property-removed] at head/openapi/specs/v0.yaml	
	in API GET /api/v3/enrollments/
		removed the required property '/items/run/allOf[#/components/schemas/CourseRunWithCourseV3]/course/allOf[#/components/schemas/CourseV3]/include_in_learn_catalog' from the response with the '200' status

error	[response-required-property-removed] at head/openapi/specs/v0.yaml	
	in API POST /api/v3/enrollments/
		removed the required property 'run/allOf[#/components/schemas/CourseRunWithCourseV3]/course/allOf[#/components/schemas/CourseV3]/include_in_learn_catalog' from the response with the '201' status

error	[response-required-property-removed] at head/openapi/specs/v0.yaml	
	in API GET /api/v3/enrollments/{id}/
		removed the required property 'run/allOf[#/components/schemas/CourseRunWithCourseV3]/course/allOf[#/components/schemas/CourseV3]/include_in_learn_catalog' from the response with the '200' status



## Changes for v1.yaml:
11 changes: 11 error, 0 warning, 0 info
error	[response-required-property-removed] at head/openapi/specs/v1.yaml	
	in API GET /api/v2/pages/?fields=*&type=cms.programpage
		removed the required property 'items/items/program_details/programs' from the response with the '200' status

error	[response-required-property-removed] at head/openapi/specs/v1.yaml	
	in API GET /api/v2/pages/{id}/
		removed the required property '/oneOf[#/components/schemas/ProgramPageItem]/program_details/programs' from the response with the '200' status

error	[response-required-property-removed] at head/openapi/specs/v1.yaml	
	in API GET /api/v2/program_certificates/{cert_uuid}/
		removed the required property 'program/programs' from the response with the '200' status

error	[response-required-property-removed] at head/openapi/specs/v1.yaml	
	in API GET /api/v2/program_enrollments/
		removed the required property '/items/program/programs' from the response with the '200' status

error	[response-required-property-removed] at head/openapi/specs/v1.yaml	
	in API DELETE /api/v2/program_enrollments/{id}/
		removed the required property '/items/program/programs' from the response with the '200' status

error	[response-required-property-removed] at head/openapi/specs/v1.yaml	
	in API GET /api/v2/program_enrollments/{id}/
		removed the required property 'program/programs' from the response with the '200' status

error	[response-required-property-removed] at head/openapi/specs/v1.yaml	
	in API GET /api/v2/programs/
		removed the required property 'results/items/programs' from the response with the '200' status

error	[response-required-property-removed] at head/openapi/specs/v1.yaml	
	in API GET /api/v2/programs/{id}/
		removed the required property 'programs' from the response with the '200' status

error	[response-required-property-removed] at head/openapi/specs/v1.yaml	
	in API GET /api/v3/enrollments/
		removed the required property '/items/run/allOf[#/components/schemas/CourseRunWithCourseV3]/course/allOf[#/components/schemas/CourseV3]/include_in_learn_catalog' from the response with the '200' status

error	[response-required-property-removed] at head/openapi/specs/v1.yaml	
	in API POST /api/v3/enrollments/
		removed the required property 'run/allOf[#/components/schemas/CourseRunWithCourseV3]/course/allOf[#/components/schemas/CourseV3]/include_in_learn_catalog' from the response with the '201' status

error	[response-required-property-removed] at head/openapi/specs/v1.yaml	
	in API GET /api/v3/enrollments/{id}/
		removed the required property 'run/allOf[#/components/schemas/CourseRunWithCourseV3]/course/allOf[#/components/schemas/CourseV3]/include_in_learn_catalog' from the response with the '200' status



## Changes for v2.yaml:
11 changes: 11 error, 0 warning, 0 info
error	[response-required-property-removed] at head/openapi/specs/v2.yaml	
	in API GET /api/v2/pages/?fields=*&type=cms.programpage
		removed the required property 'items/items/program_details/programs' from the response with the '200' status

error	[response-required-property-removed] at head/openapi/specs/v2.yaml	
	in API GET /api/v2/pages/{id}/
		removed the required property '/oneOf[#/components/schemas/ProgramPageItem]/program_details/programs' from the response with the '200' status

error	[response-required-property-removed] at head/openapi/specs/v2.yaml	
	in API GET /api/v2/program_certificates/{cert_uuid}/
		removed the required property 'program/programs' from the response with the '200' status

error	[response-required-property-removed] at head/openapi/specs/v2.yaml	
	in API GET /api/v2/program_enrollments/
		removed the required property '/items/program/programs' from the response with the '200' status

error	[response-required-property-removed] at head/openapi/specs/v2.yaml	
	in API DELETE /api/v2/program_enrollments/{id}/
		removed the required property '/items/program/programs' from the response with the '200' status

error	[response-required-property-removed] at head/openapi/specs/v2.yaml	
	in API GET /api/v2/program_enrollments/{id}/
		removed the required property 'program/programs' from the response with the '200' status

error	[response-required-property-removed] at head/openapi/specs/v2.yaml	
	in API GET /api/v2/programs/
		removed the required property 'results/items/programs' from the response with the '200' status

error	[response-required-property-removed] at head/openapi/specs/v2.yaml	
	in API GET /api/v2/programs/{id}/
		removed the required property 'programs' from the response with the '200' status

error	[response-required-property-removed] at head/openapi/specs/v2.yaml	
	in API GET /api/v3/enrollments/
		removed the required property '/items/run/allOf[#/components/schemas/CourseRunWithCourseV3]/course/allOf[#/components/schemas/CourseV3]/include_in_learn_catalog' from the response with the '200' status

error	[response-required-property-removed] at head/openapi/specs/v2.yaml	
	in API POST /api/v3/enrollments/
		removed the required property 'run/allOf[#/components/schemas/CourseRunWithCourseV3]/course/allOf[#/components/schemas/CourseV3]/include_in_learn_catalog' from the response with the '201' status

error	[response-required-property-removed] at head/openapi/specs/v2.yaml	
	in API GET /api/v3/enrollments/{id}/
		removed the required property 'run/allOf[#/components/schemas/CourseRunWithCourseV3]/course/allOf[#/components/schemas/CourseV3]/include_in_learn_catalog' from the response with the '200' status

Unexpected changes? Ensure your branch is up-to-date with main (consider rebasing).

jkachel

LGTM 👍

pdpinch · 2026-03-22T23:11:04Z

Can we also change the "MITxOnline" text that appears in the receipt? If that should be a separate PR, please open an issue.

Anas12091101 · 2026-03-24T09:57:41Z

@pdpinch, I have changed the text in the same PR. Please check the updated screenshot of the Receipt email.

pdpinch · 2026-03-24T12:36:55Z

screenshots look good to me. Can we merge this today?

Anas12091101 · 2026-03-24T12:58:42Z

The OpenAPI check is failing, but it doesn't seem like it's related to any of our changes, as we only changed the template files in this PR. I am merging this PR.

feat: replace MITx Online logo with MIT Learn logo in all emails

bf4ddae

Anas12091101 added the Needs Review label Mar 19, 2026

jkachel self-assigned this Mar 19, 2026

jkachel approved these changes Mar 19, 2026

View reviewed changes

pdpinch assigned Anas12091101 Mar 22, 2026

arslanashraf7 added Waiting on Author and removed Needs Review labels Mar 24, 2026

feat: replace MITxOnline with MIT Learn on receipt

f31b693

Anas12091101 merged commit f565d5c into main Mar 24, 2026
9 of 12 checks passed

Anas12091101 deleted the anas/change-email-logo branch March 24, 2026 12:59

odlbot mentioned this pull request Mar 24, 2026

Release 1.143.1 #3417

Merged

5 tasks

Anas12091101 mentioned this pull request Mar 31, 2026

fix: support both formats of course id for UAI course #3441

Merged

2 tasks

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat: replace MITx Online logo with MIT Learn logo in all emails#3400

feat: replace MITx Online logo with MIT Learn logo in all emails#3400
Anas12091101 merged 2 commits intomainfrom
anas/change-email-logo

Anas12091101 commented Mar 19, 2026 •

edited

Loading

Uh oh!

github-actions bot commented Mar 19, 2026 •

edited

Loading

Uh oh!

jkachel left a comment

Uh oh!

pdpinch commented Mar 22, 2026

Uh oh!

Anas12091101 commented Mar 24, 2026

Uh oh!

pdpinch commented Mar 24, 2026

Uh oh!

Anas12091101 commented Mar 24, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Conversation

Anas12091101 commented Mar 19, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

What are the relevant tickets?

Description (What does it do?)

Screenshots (if appropriate):

How can this be tested?

Additional Context

Uh oh!

github-actions bot commented Mar 19, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

OpenAPI Changes

Uh oh!

jkachel left a comment

Choose a reason for hiding this comment

Uh oh!

pdpinch commented Mar 22, 2026

Uh oh!

Anas12091101 commented Mar 24, 2026

Uh oh!

pdpinch commented Mar 24, 2026

Uh oh!

Anas12091101 commented Mar 24, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Anas12091101 commented Mar 19, 2026 •

edited

Loading

github-actions bot commented Mar 19, 2026 •

edited

Loading