forked from openedx/edx-platform
-
Notifications
You must be signed in to change notification settings - Fork 14
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
Hantkovskyi/lint fix #2567
Closed
Closed
Hantkovskyi/lint fix #2567
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Fixed the Course Card Image is absent on FB post after sharing by user. Commit for Palm release.
This is a security fix that restricts capabilities to author libraries in studio to course authors/ members of an organization. Security advisory: GHSA-3q74-3rfh-g37j
…32701) This is a backport from master: openedx#32628
Discussion: - openedx/wg-build-test-release#276 Co-authored-by: Edward Zarecor <[email protected]>
* refactor: mfe_context response to serialize object keys to camelcase (openedx#31930) * test: add tests for MFE Context API serializser (openedx#32179) * fix: empty pipelineUserDetail object pipelineUserDetail object should be empty when pipeline is not running --------- Co-authored-by: Shahbaz Shabbir <[email protected]>
…dx#33150) This is a backport from the master branch: openedx#30954 The course is visible on the main page right after creation. So anonymous users can see them and access the course about page for the courses without valid data (e.g. they will see the default course overview) When courses list filtering is processed it checks the `see_exists` permission for the anonymous user. Actually, `see_exists` means `can_load` OR `can_enroll`. `can_load` is False in our case because the course start in the future. But `can_enroll` returns True because the course's enrollment_start and enrollment_end dates are blank: ``` enrollment_start = courselike.enrollment_start or datetime.min.replace(tzinfo=UTC) enrollment_end = courselike.enrollment_end or datetime.max.replace(tzinfo=UTC) if enrollment_start < now < enrollment_end: debug("Allow: in enrollment period") return ACCESS_GRANTED ``` Set the enrollment_start the same as a course start by default if the CREATE_COURSE_WITH_DEFAULT_ENROLLMENT_START_DATE feature toggle is enabled (default is False).
Co-authored-by: Edward Zarecor <[email protected]>
Co-authored-by: Usama Sadiq <[email protected]>
…e platform/microsites
… site instead of default, fix logo path for templates
…and "See the cheatsheet for help. "
Plus fix for Hippo template
…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]>
Changes: - add quality stage - add commit convention stage
fix: [RGOeX-857] fix text mistakes on the cookie policy page
YT: https://youtrack.raccoongang.com/issue/RGOeX-284 Use this command to update discovery programs related configurations for the LMS. As a result, CatalogIntegration, ProgramsApiConfig and SiteConfiguration should be updated/created. Usage: - ./manage.py lms enable_programs --site-domain=example.com --discovery-api-url=http://edx.devstack.discovery:18381/api/v1/ --settings=production - ./manage.py lms enable_programs --site-domain=example.com --discovery-api-url=http://edx.devstack.discovery:18381/api/v1/ --service-username=discovery_worker --settings=production - Fixed overriding for ProgramsApiConfig marketing path - ProgramsApiConfig doesn’t have the marketing path by default - Removed the `--site-domain` arg, updating site configurations for all sites instead YT: - https://youtrack.raccoongang.com/issue/RGOeX-548
build: palm-rg.2 release See merge request products/rg-oex/edx-platform!178
docs: update the mr template with dor See merge request owlox-team/productsforge/oex/edx-platform!91
Use the new Django 3.2.24 with the latest security patches: https://www.djangoproject.com/weblog/2024/feb/06/security-releases/ YT: - https://youtrack.raccoongang.com/issue/RGOeX-26364
build: upgrade django version See merge request owlox-team/productsforge/oex/edx-platform!92
forge-products sync See merge request products/rg-oex/edx-platform!180
feat: [AsPu-73] Add login override See merge request hippoteam/aspen-courses/edx-platform!2
fix: [AsPu-235] The user can't log in using ACS credentials See merge request hippoteam/aspen-courses/edx-platform!3
feat: [AsPu-202] Sync user entitlements with Open edX system See merge request hippoteam/aspen-courses/edx-platform!4
fix: [AsPu-249] 500 error message when trying to login to LMS with new user See merge request hippoteam/aspen-courses/edx-platform!5
feat: apply override for discussions and allow file upload on forum See merge request hippoteam/aspen-courses/edx-platform!6
patch: add missing function import in certificate template (openedx#33904) See merge request hippoteam/aspen-courses/edx-platform!7
This is an experimental approach to introduce a role which has all Course Staff permissions, except for the Studio access. Co-authored-by: 0x29a <[email protected]>
fix: cohorts fix and max threads in inline discussions fix See merge request hippoteam/aspen-courses/edx-platform!8
andrii-hantkovskyi
force-pushed
the
hantkovskyi/lint-fix
branch
from
May 29, 2024 19:05
827c975
to
d49dd62
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.