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

Provide logo URL from the backend #25994

Merged
merged 3 commits into from
Jan 26, 2021

Conversation

awaisdar001
Copy link
Contributor

@awaisdar001 awaisdar001 commented Jan 6, 2021

Provide Email Logo URL from Backend

This PR adds a new method in the branding API which would be used to serve a logo URL from embedding into emails. This method is used in Bulk Email and edX ACE emails.

Once that is merged, the base template in Django admin needs to use the context variable logo_url.

TNL-7872

@awaisdar001
Copy link
Contributor Author

jenkins run js

@awaisdar001
Copy link
Contributor Author

jenkins run quality

@edx-status-bot
Copy link

Your PR has finished running tests. There were no failures.

Copy link
Contributor

@saadyousafarbi saadyousafarbi left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@awaisdar001 awaisdar001 merged commit eeed906 into master Jan 26, 2021
@awaisdar001 awaisdar001 deleted the aj/bulk-email-provide-logo-from-bkend branch January 26, 2021 10:02
@edx-pipeline-bot
Copy link
Contributor

EdX Release Notice: This PR has been deployed to the staging environment in preparation for a release to production.

@edx-pipeline-bot
Copy link
Contributor

EdX Release Notice: This PR may have caused e2e tests to fail on Stage. If you're a member of the edX org, please visit #e2e-troubleshooting on Slack to help diagnose the cause of these failures. Otherwise, it is the reviewer's responsibility. E2E tests have failed. https://gocd.tools.edx.org/go/tab/pipeline/history/deploy_to_stage

@edx-pipeline-bot
Copy link
Contributor

EdX Release Notice: This PR has been deployed to the production environment.

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.

6 participants