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

Niedielnitsev ivan/offline problem media #2565

Closed

Conversation

NiedielnitsevIvan
Copy link

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 26 commits May 27, 2024 14:03
* 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
* refactor: [AXM-475] refactor firebase settings

* fix: [AXM-475] fix device token API permissions

* fix: [AXM-475] change texts for push notifs

* fix: [AXM-475] update django-push-notifications version

* refactor: [AXM-475] refactor firebase settings
…eature/Add-new-events-to-send-push-notifications-for-all-responses-and-comments-in-the-thread

feat: [AXM-556] refactor discussion push notifications sending
(cherry picked from commit 15ef236cdc3eb05da120449b8831b0d77aebb5bf)
(cherry picked from commit 4a388b574c8a44e999514950aae02cbd96987686)
@NiedielnitsevIvan NiedielnitsevIvan self-assigned this May 29, 2024
@NiedielnitsevIvan NiedielnitsevIvan marked this pull request as draft May 29, 2024 15:59
@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.

4 participants