Skip to content

feat: order creation for mobile IAP #400

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 16 commits into from
May 13, 2025
Merged

feat: order creation for mobile IAP #400

merged 16 commits into from
May 13, 2025

Conversation

NawfalAhmed
Copy link
Member

@NawfalAhmed NawfalAhmed commented May 6, 2025 •
edited
Loading

Description

  • mobile purchase via CC

@NawfalAhmed NawfalAhmed self-assigned this May 6, 2025
@NawfalAhmed NawfalAhmed changed the title feat: create order for mobile (#397) feat: order creation for mobile IAP May 6, 2025
@NawfalAhmed NawfalAhmed mentioned this pull request May 6, 2025
Closed
6 tasks
@NawfalAhmed NawfalAhmed force-pushed the mobile_iap branch 2 times, most recently from c807acb to 8bf4400 Compare May 7, 2025 15:12
NawfalAhmed and others added 5 commits May 7, 2025 20:18
@NawfalAhmed
feat: create order for mobile (#397)
a75cd7e 
* feat: learner-specific checkout api is created

* feat: add prepare cart

* feat: use single api

* test: add unit tests

* fix: lint

---------

Co-authored-by: Atiya Ishaq <atiya.ishaq@A006-01676.local>
@NawfalAhmed
feat: updates and fixes
adb1f92
@NawfalAhmed
fix: more updates, add standalone price
0eb7d55
@NawfalAhmed
fix: lint
d61df97
@NawfalAhmed
docs: updated docs
8887dd4
abdullahwaheed
abdullahwaheed approved these changes May 8, 2025
View reviewed changes
Copy link
Contributor

@abdullahwaheed abdullahwaheed left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some minor changes. Overall looks good

@github-actions GitHub Actions
Copy link

github-actions bot commented May 13, 2025 •
edited
Loading

Coverage report

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)		Lines missing
  commerce_coordinator
  urls.py
  commerce_coordinator/apps/commercetools
  clients.py 166, 800-801, 994-1000, 1048-1054, 1074-1087, 1103-1110, 1125-1131, 1164-1170, 1250-1256, 1291-1298, 1385-1391, 1421-1428
  http_api_client.py
  commerce_coordinator/apps/commercetools/catalog_info
  constants.py
  commerce_coordinator/apps/commercetools/tests
  conftest.py
  test_clients.py
  test_http_api_client.py
  commerce_coordinator/apps/iap
  serializers.py
  urls.py
  utils.py
  views.py 68
  commerce_coordinator/apps/iap/tests
  test_utils.py
  test_views.py
  commerce_coordinator/apps/lms
  views.py
  commerce_coordinator/apps/lms/tests
  test_views.py
Project Total  

This report was generated by python-coverage-comment-action

@NawfalAhmed NawfalAhmed merged commit 7372e31 into main May 13, 2025
8 checks passed
@NawfalAhmed NawfalAhmed deleted the mobile_iap branch May 13, 2025 13:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@abdullahwaheed abdullahwaheed abdullahwaheed approved these changes

@shafqatfarhan shafqatfarhan Awaiting requested review from shafqatfarhan

Assignees

@NawfalAhmed NawfalAhmed

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@NawfalAhmed @abdullahwaheed