-
Notifications
You must be signed in to change notification settings - Fork 10
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
perf: add palm support DS-703 #254
Conversation
9cc5cda
to
4fe1d00
Compare
924de29
to
a9ccb3d
Compare
The tests were passed. It starts to fail when I update the readme for the length of the PR. For now, I will put the labeler fail in false, but if you think it is too long, I will split the readme update in another PR. |
0822b7d
to
1b18997
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Everything is working as expected!
I left a little comment in the test cases doc, but nothing else.
@MaferMazu, Is it possible to rebase the branch to update it, instead of generating a merge commit? |
…ed by backports-zoneinfo
bb78b9d
to
1217c15
Compare
@dcoa ready! |
Description
This migration PR does the following:
eox-theming
migration)backports-zoneinfo
, which causes errors. (Note: that package is in edx-platform too: https://github.com/openedx/edx-platform/blob/open-release/palm.master/requirements/edx/base.txt#L54)social-auth-core
to use social_core without try exceptsTesting instructions
Additional information
Related PR: #257