Skip to content

Conversation

@github-actions
Copy link

Ref: #914

svillegas-cdd and others added 5 commits October 15, 2025 10:57
- This change addresses cases where multiple CSV rows with empty fields appeared,
representing additional impuestos linked to the same folio as the main row.
- Replaced individual fields (`codigo_otro_impuesto`, `valor_otro_impuesto`, `tasa_otro_impuesto`) with unified `otros_impuestos` structure.
- Updated schemas, data models, and deserializers to handle "Otros Impuestos".
- Adjusted tests and test data to validate new structure and edge cases.

Ref: https://app.shortcut.com/cordada/story/16788/
…ors-otros-impuestos

rcv: Refactor and extend parsing for "Otros Impuestos" field
@github-actions github-actions bot added task Task or chore kind: deploy Deployment labels Oct 20, 2025
@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
14.2% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

@svillegas-cdd svillegas-cdd marked this pull request as ready for review October 20, 2025 17:54
@svillegas-cdd svillegas-cdd requested a review from a team as a code owner October 20, 2025 17:54
@svillegas-cdd svillegas-cdd requested review from a team and jtrobles-cdd and removed request for a team October 20, 2025 17:54
@svillegas-cdd svillegas-cdd merged commit aa198ff into master Oct 20, 2025
24 of 25 checks passed
@svillegas-cdd svillegas-cdd deleted the deploy/v0.60.0 branch October 20, 2025 19:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind: deploy Deployment task Task or chore

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants