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

chore: Syncing 2u/project-theseus with master [2023-08-29] #4022

Merged

Conversation

grmartin
Copy link

chore: Syncing 2u/project-theseus with master [2023-08-29]

This is a regular sync between the master (@ b5a96f1) branch of openedx/ecommerce with 2u/project-theseus

feanil and others added 30 commits March 1, 2023 14:04
The .github/workflows/self-assign-issue.yml workflow is missing or needs an update to stay in
sync with the current standard for this workflow as defined in the
`.github` repo of the `openedx` GitHub org.
…yml`.

The .github/workflows/add-remove-label-on-comment.yml workflow is missing or needs an update to stay in
sync with the current standard for this workflow as defined in the
`.github` repo of the `openedx` GitHub org.
…d.yml`.

The .github/workflows/add-depr-ticket-to-depr-board.yml workflow is missing or needs an update to stay in
sync with the current standard for this workflow as defined in the
`.github` repo of the `openedx` GitHub org.
We now have a org wide CONTRIBUTING.md that points to our correct
general contributing guidelines.  We don't need repo specific ones that
forward to other contributing docs.
* fix: Pick the right purchase from ios response

iOS response contain multiple purchases, instead of picking the first purchase,
pick the one which have given product id and latest date.
LEARNER-9261
* feat: Added Android refund api

Like Apple android doesn't have callback for every refund. Therefore we have created an endpoint  which we will hit daily through ecommerce worker.
Learner-9149
* feat: Error if products in basket are already purchased

* refactor: Add tests, Improve error message

* refactor: Update docstring

* test: Increase coverage
* fix: Fix error in checkout api for mobile
* fix: Return error in case of duplicate transactionID for mobile

* refactor: Review feedback, add documentation
…hboard (#3938)

Forked catalogue app from oscar and added course and expire field in ProductForm. This change will enable to
add Android sku from a same dashboard page.
Reordered the JWT decoders to first use the standard
library version, and then use the custom ecommerce
decoder which uses multiple issuers. In this way, we
can see if any JWTs cannot be decoded by that standard
library version, and when and if we are ready to retire
the custom JWT decoding code.

See DEPR openedx/public-engineering#83
Monitoring features such as use of the increment
method, to increment a custom attribute, requires
the CachedCustomMonitoringMiddleware. This has been
added so the earlier calls to increment will function.
Add monitoring for the discount JWT.
Co-authored-by: IrfanUddinAhmad <irfanahmad@arbisoft.com>
The edx-sphinx theme is being deprecated, and replaced with sphinx-book-theme.
This removes references to the deprecated theme and replaces them with the new
standard theme for the platform.
* test: Add tests for mobile In-app purchases

This reverts commit 54ea975.
Codecov PyPI package was removed on 12 April and the recommended step is to migrate to codecov Github Action instead.
…T-7059

Also removes codecov from ci.yml workflow.
We're moving towards a single set of guidelines org-wide.
* feat!: remove custom JWT decoding

Removes the ecommerce custom JWT decoding, and replaces
with the simple decoding from the edx-drf-extensions
library.

* fix: drop constraints and make upgrade

* fix: handle major upgrade of django-crispy-forms

The major upgrade of django-crispy-forms called for
some changes related to bootstrap3 and dependencies.
See https://github.com/django-crispy-forms/django-crispy-forms/blob/main/CHANGELOG.md#major-changes-and-migration-guide

* fix: code coverage reporting

Codecov no longer exists on PyPI, so switch to github
action to run coverage report.

---------

Co-authored-by: Muhammad Zubair <syedzubairtahir12@gmail.com>
…3950)

* fix: Course to have multiple seats with certificate_type attribute
* refactor: Modify SKU generation hash, add tests
* test: Modify tests
Jenkins job for building devstack images is temporarily broken.
This should fix the devstack settings until this configuration
change lands in an updated image:
openedx-unsupported/configuration#6921
moeez96 and others added 25 commits May 11, 2023 10:30
* refactor: Improve exception handling for mobile IAP
* refactor: pylint fixes
* feat: Fix capture_context error on Payment MFE

* feat: removed whitespace

* feat: removed whitespace

* feat: modified test case

* feat: modified test case

---------

Co-authored-by: Muhammad Zubair <syedzubairtahir12@gmail.com>
….3.3) (#3968)

Co-authored-by: Muhammad Zubair <syedzubairtahir12@gmail.com>
Will create a Jira ticket instead of tagging all the members of
@openedx/revenue-squad.
* feat: add endpoint to run SDN check and return counts

* feat: add SDNCheckFailure REST APi
* fix: fix 500 on SDN for subscriptions
* feat: Store price and currency for mobile IAP
* feat: unenroll refunded android users daily

Django management command to un-enroll refunded android users. This command will be run by Jenkins job daily.
* feat: mail mobile team for a mobile course change in publisher

This will fix any unknown change from publisher to a course having mobile seats.
After this fix mobile team will see mail and adjust price of the course on playstore or appstore.
In the longer run we want to replace this solution by changing the course price directly using mobile platform apis.

LEARNER-9377

* fix: fixed coverage issue
This is a regular sync between the master (@ b5a96f1) branch of openedx/ecommerce with 2u/project-theseus
@grmartin grmartin self-assigned this Aug 29, 2023
@grmartin grmartin merged commit 8a80bdf into 2u/project-theseus Aug 29, 2023
8 of 10 checks passed
@grmartin grmartin deleted the 2u/project-theseus_pr/grmartin/resync_2023-08-29 branch August 29, 2023 14:13
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.

None yet