-
Notifications
You must be signed in to change notification settings - Fork 1
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
Palm minor version upgrade and fixes #43
Conversation
Fixed the Course Card Image is absent on FB post after sharing by user. Commit for Palm release.
…a256` (openedx#33345) --------- Co-authored-by: Muhammad Soban Javed <[email protected]> Co-authored-by: Robert Raposa <[email protected]> Co-authored-by: Muhammad Soban Javed <[email protected]>
fix: share certificate in FB
…t-palm fix: User retirement 404 for state RETIRING_FORUMS
[Palm-Backport] Update DEFAULT_HASHING_ALGORITHM to sha256
Co-authored-by: Travis McKinney <[email protected]>
propagate override
…dx-platform into _medality-palm-travis
…tform into _medality-palm-travis
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.
One comment below. Have you confirmed that any recent changes (such as Patrick's) to medality-olive have been pulled in?
@@ -846,8 +847,13 @@ def is_enrolled(cls, user, course_key): | |||
|
|||
`course_id` is our usual course_id string (e.g. "edX/Test101/2013_Fall) | |||
""" | |||
enrollment_state = cls._get_enrollment_state(user, course_key) | |||
return enrollment_state.is_active or False | |||
try: |
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.
Should we tag this with @medality_custom
?
@jcohen28 added the medality_custom tag and confirmed this branch is up to date with medality-olive! |
No description provided.