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: only show warning for subcontracting receipt #2652

Merged
merged 4 commits into from
Oct 14, 2024

Conversation

Ninad1306
Copy link
Contributor

@Ninad1306 Ninad1306 commented Oct 7, 2024

For subcontracting, users could have different workflows:

eg: Material sent to subcontractor directly from the supplier's warehouse.
Because of these reasons, it becomes difficult to add references.

With a warning, this document can be skipped in ITC-04.

Huly®: IC-2776

@vorasmit
Copy link
Member

vorasmit commented Oct 7, 2024

Can we add test cases for them?

Copy link

codecov bot commented Oct 14, 2024

Codecov Report

Attention: Patch coverage is 76.47059% with 4 lines in your changes missing coverage. Please review.

Project coverage is 61.34%. Comparing base (a113675) to head (d9a1862).
Report is 1 commits behind head on develop.

Files with missing lines Patch % Lines
.../gst_india/overrides/subcontracting_transaction.py 76.47% 4 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##           develop    #2652      +/-   ##
===========================================
+ Coverage    60.41%   61.34%   +0.92%     
===========================================
  Files          119      119              
  Lines        11493    11506      +13     
===========================================
+ Hits          6944     7058     +114     
+ Misses        4549     4448     -101     
Files with missing lines Coverage Δ
india_compliance/hooks.py 100.00% <ø> (ø)
.../gst_india/overrides/subcontracting_transaction.py 80.00% <76.47%> (+50.93%) ⬆️

... and 1 file with indirect coverage changes

Impacted file tree graph

@vorasmit vorasmit merged commit ad4e139 into resilient-tech:develop Oct 14, 2024
11 checks passed
mergify bot pushed a commit that referenced this pull request Oct 14, 2024
* fix: only show warning for subcontracting receipt

* chore: better formatting

* test: test case to validate doc references

---------

(cherry picked from commit ad4e139)

# Conflicts:
#	india_compliance/gst_india/overrides/test_subcontracting_transaction.py
@Ninad1306 Ninad1306 deleted the remove_validation branch October 14, 2024 10:06
mergify bot added a commit that referenced this pull request Oct 14, 2024
…tfix/pr-2652

fix: only show warning for subcontracting receipt (backport #2652)
india-compliance-bot added a commit that referenced this pull request Oct 23, 2024
## [15.13.5](v15.13.4...v15.13.5) (2024-10-23)

### Bug Fixes

* add comment in gst return log for backdated transactions ([#2656](#2656)) ([49bf757](49bf757))
* apply css to specific fields ([4021e06](4021e06))
* changes for test case ([943b0c7](943b0c7))
* changing the sequence as per sales invoice ([c886931](c886931))
* connection for e-Invoice log in purchase invoice ([5c70350](5c70350))
* filter for invalid invoice number in document issue summary ([#2641](#2641)) ([62912b6](62912b6)), closes [#2608](#2608)
* only show warning for subcontracting receipt ([#2652](#2652)) ([9536376](9536376))
* pos based on account settings ([#2647](#2647)) ([f6a4a9c](f6a4a9c))
* resolve merge conflict ([c457198](c457198))
* resolve merge conflict ([fb1cfac](fb1cfac))
* resolved conflicts ([10dcb49](10dcb49))
* update bill_from_address when required ([36180f5](36180f5))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants