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

fix: enqueue purchase reconcilliation #2372

Draft
wants to merge 6 commits into
base: develop
Choose a base branch
from

Conversation

Sanket322
Copy link
Contributor

closes: #1544

Copy link

codecov bot commented Jul 9, 2024

Codecov Report

Attention: Patch coverage is 17.64706% with 28 lines in your changes missing coverage. Please review.

Project coverage is 59.60%. Comparing base (60b986e) to head (2a025fa).
Report is 6 commits behind head on develop.

Files Patch % Lines
...compliance/gst_india/report/icutility/icutility.py 0.00% 16 Missing ⚠️
...econciliation_tool/purchase_reconciliation_tool.py 33.33% 12 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##           develop    #2372      +/-   ##
===========================================
- Coverage    61.42%   59.60%   -1.83%     
===========================================
  Files          118      119       +1     
  Lines        11282    11370      +88     
===========================================
- Hits          6930     6777     -153     
- Misses        4352     4593     +241     
Files Coverage Δ
...econciliation_tool/purchase_reconciliation_tool.py 22.45% <33.33%> (-1.67%) ⬇️
...compliance/gst_india/report/icutility/icutility.py 0.00% <0.00%> (ø)

... and 5 files with indirect coverage changes

Impacted file tree graph

@ljain112
Copy link
Collaborator

It will require something like GSTR-1 Beta where data is saved in JSON format

@vorasmit
Copy link
Member

@vorasmit vorasmit marked this pull request as draft July 29, 2024 09:25
@Sanket322 Sanket322 force-pushed the enqueue_purchase_reconciliation branch 2 times, most recently from 1417999 to a47cb51 Compare August 7, 2024 12:48
@Sanket322 Sanket322 marked this pull request as ready for review August 7, 2024 13:00
@Sanket322 Sanket322 force-pushed the enqueue_purchase_reconciliation branch 4 times, most recently from 9c6e411 to c6deb5f Compare August 10, 2024 10:41
@Sanket322 Sanket322 force-pushed the enqueue_purchase_reconciliation branch from c6deb5f to 5070a9c Compare August 10, 2024 10:53
@Sanket322 Sanket322 marked this pull request as draft August 12, 2024 11:33
@Sanket322 Sanket322 assigned Sanket322 and unassigned Sanket322 Aug 14, 2024
@Sanket322 Sanket322 requested a review from vorasmit August 14, 2024 07:22
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.

enqueue purchase reconcilliation
3 participants