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 invoice copy_data to dont copy mail info #818

Merged
merged 1 commit into from
Feb 25, 2025
Merged

Conversation

anxodio
Copy link
Contributor

@anxodio anxodio commented Feb 25, 2025

Objectiu

https://freescout.somenergia.coop/conversation/7976387?folder_id=87

S'estava copiant tota la info del mail i ens molestava en les abonadores, que quedaven linkades al mail de la factura original i per tant al imprimir el PDF agafava també l'original.

Comprovacions

  • Hi ha testos
  • Reiniciar serveis
  • Actualitzar mòdul
  • Script de migració
  • Modifica traduccions

Copy link

Test Results

  2 files  ± 0    2 suites  ±0   8s ⏱️ +8s
162 tests +87  161 ✅ +86  1 💤 +1  0 ❌ ±0 
162 runs  +86  161 ✅ +85  1 💤 +1  0 ❌ ±0 

Results for commit 982026f. ± Comparison against base commit fcf35ce.

This pull request removes 3 and adds 90 tests. Note that renamed tests count towards both.
destral.testing.OOBaseTests ‑ test_access_rules
destral.testing.OOBaseTests ‑ test_all_views
destral.testing.OOBaseTests ‑ test_translate_modules
Misc ‑ when testing if the request is from webclient
Total ‑ Misc
estral.testing.OOBaseTests.test_access_rule ‑ destral.testing.OOBaseTests.test_access_rules
estral.testing.OOBaseTests.test_all_view ‑ destral.testing.OOBaseTests.test_all_views
estral.testing.OOBaseTests.test_translate_module ‑ destral.testing.OOBaseTests.test_translate_modules
iscedata_facturacio_comer_som.tests.tests_FacturacioFacturaReport_some.Tests_FacturacioFacturaReport_amount_destination.test__som_report_comp_amount_destination__with_altres_line ‑ giscedata_facturacio_comer_som.tests.tests_FacturacioFacturaReport_some.Tests_FacturacioFacturaReport_amount_destination.test__som_report_comp_amount_destination__with_altres_lines
iscedata_facturacio_comer_som.tests.tests_FacturacioFacturaReport_some.Tests_FacturacioFacturaReport_amount_destination.test__som_report_comp_amount_destination__without_altres_line ‑ giscedata_facturacio_comer_som.tests.tests_FacturacioFacturaReport_some.Tests_FacturacioFacturaReport_amount_destination.test__som_report_comp_amount_destination__without_altres_lines
iscedata_facturacio_comer_som.tests.tests_FacturacioFacturaReport_some.Tests_FacturacioFacturaReport_company_component.test__som_report_comp_company__bas ‑ giscedata_facturacio_comer_som.tests.tests_FacturacioFacturaReport_some.Tests_FacturacioFacturaReport_company_component.test__som_report_comp_company__base
iscedata_facturacio_comer_som.tests.tests_FacturacioFacturaReport_some.Tests_FacturacioFacturaReport_company_component.test__som_report_comp_company__so ‑ giscedata_facturacio_comer_som.tests.tests_FacturacioFacturaReport_some.Tests_FacturacioFacturaReport_company_component.test__som_report_comp_company__som
iscedata_facturacio_comer_som.tests.tests_FacturacioFacturaReport_some.Tests_FacturacioFacturaReport_contract_data_component.test_som_report_comp_contract_data_20A_amb_aut ‑ giscedata_facturacio_comer_som.tests.tests_FacturacioFacturaReport_some.Tests_FacturacioFacturaReport_contract_data_component.test_som_report_comp_contract_data_20A_amb_auto
…

@pauboixsom pauboixsom merged commit 8fcffe5 into main Feb 25, 2025
6 checks passed
@pauboixsom pauboixsom deleted the fix_abonament_mail branch February 25, 2025 13:55
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.

3 participants