Skip to content

Conversation

@jtrobles-cdd
Copy link
Member

@jtrobles-cdd jtrobles-cdd commented Dec 2, 2025

When the "tipo DTE" of a RcvDetalleEntry is a "nota" (e.g. "nota de crédito electrónica"), the as_dte_data_l2() method throws an exception because it does not know how to determine the emisor and receptor of the DTE.

Ref: #949

@jtrobles-cdd jtrobles-cdd self-assigned this Dec 2, 2025
@jtrobles-cdd jtrobles-cdd requested a review from a team as a code owner December 2, 2025 19:43
@jtrobles-cdd jtrobles-cdd added bug Something isn't working component: rcv labels Dec 2, 2025
@jtrobles-cdd jtrobles-cdd changed the title rcv: RcvDetalleEntry.as_dte_data_l2() does not work for 'Notas' rcv: RcvDetalleEntry.as_dte_data_l2() does not work when tipo DTE is a "Nota" Dec 2, 2025
When the "tipo DTE" of a `RcvDetalleEntry` is a "nota" (e.g. "nota de crédito electrónica"),
the `as_dte_data_l2()` method throws an exception because it does not
know how to determine the `emisor` and `receptor` of the DTE.
@jtrobles-cdd jtrobles-cdd force-pushed the fix-rcv-detalle-entry-to-dtedata branch from 3b68dad to 9637330 Compare December 2, 2025 19:44
@sonarqubecloud
Copy link

sonarqubecloud bot commented Dec 2, 2025

Quality Gate Failed Quality Gate failed

Failed conditions
B Maintainability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@codecov
Copy link

codecov bot commented Dec 2, 2025

Codecov Report

❌ Patch coverage is 0% with 10 lines in your changes missing coverage. Please review.
✅ Project coverage is 88.03%. Comparing base (ae3320b) to head (9637330).
⚠️ Report is 2 commits behind head on develop.

Files with missing lines Patch % Lines
src/cl_sii/rcv/data_models.py 0.00% 10 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #952      +/-   ##
===========================================
- Coverage    88.26%   88.03%   -0.24%     
===========================================
  Files           40       40              
  Lines         3793     3803      +10     
  Branches       396      398       +2     
===========================================
  Hits          3348     3348              
- Misses         290      300      +10     
  Partials       155      155              

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@jtrobles-cdd jtrobles-cdd disabled auto-merge December 2, 2025 19:58
@jtrobles-cdd jtrobles-cdd merged commit 492547b into develop Dec 2, 2025
25 of 27 checks passed
@jtrobles-cdd jtrobles-cdd deleted the fix-rcv-detalle-entry-to-dtedata branch December 2, 2025 19:59
@jtrobles-cdd jtrobles-cdd mentioned this pull request Dec 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working component: rcv

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants