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

MyECLPay #611

Draft
wants to merge 55 commits into
base: main
Choose a base branch
from
Draft

MyECLPay #611

wants to merge 55 commits into from

Conversation

armanddidierjean
Copy link
Member

Description

Please explain the changes you made here.

Checklist

  • Created tests which fail without the change (if possible)
  • All tests passing
  • Extended the documentation, if necessary

@armanddidierjean armanddidierjean force-pushed the myeclpay branch 2 times, most recently from 47f1137 to 1777ada Compare November 17, 2024 08:04
Copy link

codecov bot commented Nov 17, 2024

Codecov Report

Attention: Patch coverage is 86.04061% with 110 lines in your changes missing coverage. Please review.

Project coverage is 81.42%. Comparing base (1d1de79) to head (a774e63).

Files with missing lines Patch % Lines
app/core/myeclpay/endpoints_myeclpay.py 76.35% 96 Missing ⚠️
app/core/myeclpay/cruds_myeclpay.py 89.83% 12 Missing ⚠️
app/core/payment/cruds_payment.py 50.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #611      +/-   ##
==========================================
+ Coverage   81.06%   81.42%   +0.35%     
==========================================
  Files         134      140       +6     
  Lines       10319    11104     +785     
==========================================
+ Hits         8365     9041     +676     
- Misses       1954     2063     +109     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Marc-Andrieu Marc-Andrieu mentioned this pull request Dec 25, 2024
4 tasks
foucblg pushed a commit that referenced this pull request Jan 6, 2025
> [!IMPORTANT]
> We really need to release new versions **soon**, as we have not for
several months and there's some time needed to test in alpha before
prod.

This goes hand in hand with the **major version bump** [for
Titan](../../Titan/pull/462).

Both PRs are ready, and _seemingly_ waiting for some more **breaking
changes** to be merged, including (by likelihood):

- [ ] Allow students from other Schools to have a MyECL account: #641 
- [ ] MyECLPay: #611 
- [ ] "_Centrale Mega Meme_" Meme module: [no PR yet] [see
branch](../tree/cmm)
- [x] External Notifications: #613
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request module payment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants