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

feat: offline problem media v2 #2559

Closed
wants to merge 21 commits into from

Conversation

monteri
Copy link
Collaborator

@monteri monteri commented May 15, 2024

Description

Describe what this pull request changes, and why. Include implications for people using this change.
Design decisions and their rationales should be documented in the repo (docstring / ADR), per
OEP-19, and can be
linked here.

Useful information to include:

  • Which edX user roles will this change impact? Common user roles are "Learner", "Course Author",
    "Developer", and "Operator".
  • Include screenshots for changes to the UI (ideally, both "before" and "after" screenshots, if applicable).
  • Provide links to the description of corresponding configuration changes. Remember to correctly annotate these
    changes.

Supporting information

Link to other information about the change, such as Jira issues, GitHub issues, or Discourse discussions.
Be sure to check they are publicly readable, or if not, repeat the information here.

Testing instructions

Please provide detailed step-by-step instructions for testing this change.

Deadline

"None" if there's no rush, or provide a specific date or event (and reason) if there is one.

Other information

Include anything else that will help reviewers and consumers understand the change.

  • Does this change depend on other changes elsewhere?
  • Any special concerns or limitations? For example: deprecations, migrations, security, or accessibility.
  • If your database migration can't be rolled back easily.

KyryloKireiev and others added 18 commits May 2, 2024 15:47
* feat: [AXM-24] Update structure for course enrollments API

* style: [AXM-24] Improve code style

* fix: [AXM-24] Fix student's latest enrollment filter
* fix: workaround for staticcollection introduced in e40a01c

* feat: [AXM-40] add courses progress to enrollment endpoint

* refactor: [AXM-40] add caching to improve performance

* refactor: [AXM-40] add progress only for primary course

* refactor: [AXM-40] refactor enrollment caching optimization

---------

Co-authored-by: Glib Glugovskiy <[email protected]>
* feat: [AXM-53] add assertions for primary course

* test: [AXM-53] fix tests

* style: [AXM-53] change future_assignment default value to None

* refactor: [AXM-53] add some optimization for assignments collecting
* feat: [AXM-24] Update structure for course enrollments API (#2515)

* feat: [AXM-24] Update structure for course enrollments API

* style: [AXM-24] Improve code style

* fix: [AXM-24] Fix student's latest enrollment filter

* feat: [AXM-47] Add course_status field to primary object (#2517)

* feat: [AXM-40] add courses progress to enrollment endpoint (#2519)

* fix: workaround for staticcollection introduced in e40a01c

* feat: [AXM-40] add courses progress to enrollment endpoint

* refactor: [AXM-40] add caching to improve performance

* refactor: [AXM-40] add progress only for primary course

* refactor: [AXM-40] refactor enrollment caching optimization

---------

Co-authored-by: Glib Glugovskiy <[email protected]>

* feat: [AXM-53] add assertions for primary course (#2522)

* feat: [AXM-53] add assertions for primary course

* test: [AXM-53] fix tests

* style: [AXM-53] change future_assignment default value to None

* refactor: [AXM-53] add some optimization for assignments collecting

* feat: [AXM-200] Implement user's enrolments status API

* style: [AXM-200] Improve code style

* refactor: [AXM-200] Divide get method into smaller methods

---------

Co-authored-by: NiedielnitsevIvan <[email protected]>
Co-authored-by: Glib Glugovskiy <[email protected]>
…uses (#2532)

* feat: [AXM-33] create enrollments filtering by course completion statuses

* test: [AXM-33] add tests for filtrations

* style: [AXM-33] fix pylint issues
… API (#2546)

* feat: [AXM-297, AXM-310] Add progress to assignments and total course progress

* feat: [AXM-297] Add progress to assignments

* style: [AXM-297] Try to fix linters (add docstrings)

* refactor: [AXM-297] Add typing, refactor methods
…ame way as past assignments (#2550)

* feat: [AXM-288] Change response to represent Future assignments the same way as past assignments

* refactor: [AXM-288] Refactor get_course_assignments

Co-authored-by: monteri <[email protected]>

* refactor: [AXM-288] Refactor get_course_assignments method

---------

Co-authored-by: monteri <[email protected]>
* feat: [AXM-252] create policy for push notifications

* feat: [AXM-252] add API for store device token

* feat: [AXM-252] add settings for edx-ace push notifications

* chore: [AXM-252] add edx-ace and django-push-notification to dev requirements

* chore: [AXM-252] update edx-ace version

* fix: [AXM-252] add create token edndpoint to urls

* chore: [AXM-252] update django push notifications version

* style: [AXM-252] fix code style issues after review

* chore: [AXM-252] bump edx-ace version

* refactor: [AXM-252] some push notif policy refactoring

* chore: [AXM-252] change edx-ace branch to mob-develop

* chore: [AXM-252] recompile requirements after rebase
* feat: [AXM-271] Add push notification event to discussions

* refactor: [AXM-252] add extra context to push notification
* feat: [AXM-287,310,331] Change course progress calculation logic

* style: [AXM-287,310,331] Remove commented code

* fix: [AXM-287,310,331] Change course assignments gather logic
…2557)

* feat: [AXM-373] add push notifications for user enroll

* feat: [AXM-373] add push notifications for user unenroll

* feat: [AXM-373] add push notifications for add course beta testers

* feat: [AXM-373] add push notifications for remove course beta testers

* style: [AXM-373] remove debugger

* fix: [AXM-373] fix after demo
(cherry picked from commit 15ef236cdc3eb05da120449b8831b0d77aebb5bf)
(cherry picked from commit 4a388b574c8a44e999514950aae02cbd96987686)
@monteri monteri force-pushed the zadorozhnii/offline-problem-media branch from 6bd6b6b to 531e6b8 Compare May 16, 2024 16:49
@monteri monteri marked this pull request as draft May 16, 2024 17:02
@GlugovGrGlib
Copy link
Collaborator

POC finished, this was done as part of feature PR

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.

5 participants