-
-
Notifications
You must be signed in to change notification settings - Fork 305
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
[16.0] [REF] Comunicazione con SdI tramite WebService (SDICoop) + allineamento generale di l10n_it_sdi_channel #3785
Conversation
Hi @sergiocorato, |
2feb259
to
55463b9
Compare
#3785 (comment) ne ho messe 4 @tafaRU 😅 |
k allora si tratta di un errore, venerdì abbiamo provato dei filtri con @francesco-ooops e @TheMule71, direi che in questo caso non ha funzionato 😉 |
fba4c28
to
d557c69
Compare
Credo risolva anche #2477. |
Generic sending/delivering fields and views are common to all channels
…r e-invoice notifications
… avoid to call receive_fe on a specific channel, that could produce security errors in multi-company
…), use flush_model(), flush_recordset() or env.flush_all() instead
…not allowed to modify 'ES channel' (sdi.channel) records.
Other changes included in OCA#3118
…uld not behave differently from UI, valid test is test_action_open_export_send_sdi_ui
AssertionError: email_exchange_system is a required field ({'required': [('channel_type', '=', 'pec')], 'invisible': [('channel_type', '!=', 'pec')]})
The signature of fatturapa.attachment.out.get_invoice_obj must be the same of fatturapa.attachment.in.get_invoice_obj because the same method is called on both objects
b981472
to
f143cf4
Compare
/ocabot merge minor |
What a great day to merge this nice PR. Let's do it! |
Congratulations, your PR was merged at ee1f87d. Thanks a lot for contributing to OCA. ❤️ |
Issues:
porting #3118 + #3207 + #3120 + #3121 + #3475