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

Permettere di ricevere notifiche per la ricezione di fatture elettroniche o notifiche #3121

Merged

Conversation

SirTakobi
Copy link
Contributor

@SirTakobi SirTakobi commented Jan 5, 2023

Implementa parzialmente #2883 per 14.0.
Implementa #3212 per 14.0.
Forward port di #2884.
Ho escluso i commit relativi al nuovo modulo l10n_it_fatturapa_sdicoop perché sono nella PR di migrazione #3122.

Include #3120.

@OCA-git-bot
Copy link
Contributor

Hi @sergiocorato,
some modules you are maintaining are being modified, check this out!

@SirTakobi SirTakobi force-pushed the 14.0-imp-l10n_it_sdi_channel-notification branch from e900c04 to f5eb003 Compare January 5, 2023 10:51
@SirTakobi SirTakobi marked this pull request as ready for review January 5, 2023 16:12
@SirTakobi SirTakobi force-pushed the 14.0-imp-l10n_it_sdi_channel-notification branch from 3604447 to 6a226cd Compare February 27, 2023 09:44
@SirTakobi SirTakobi marked this pull request as draft March 15, 2023 14:04
@SirTakobi SirTakobi force-pushed the 14.0-imp-l10n_it_sdi_channel-notification branch 3 times, most recently from 4de40a0 to 2128301 Compare March 15, 2023 15:07
@SirTakobi SirTakobi marked this pull request as ready for review March 15, 2023 15:33
@SirTakobi SirTakobi force-pushed the 14.0-imp-l10n_it_sdi_channel-notification branch from 2128301 to 4645612 Compare March 20, 2023 10:32
@SirTakobi SirTakobi force-pushed the 14.0-imp-l10n_it_sdi_channel-notification branch from 15e455d to 65ef7c9 Compare March 22, 2023 10:25
Copy link
Member

@eLBati eLBati left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Grazie

Copy link
Contributor

@mrcast mrcast left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Functional test:OK!

@OCA-git-bot
Copy link
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@eLBati
Copy link
Member

eLBati commented Mar 23, 2023

/ocabot merge minor

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 14.0-ocabot-merge-pr-3121-by-eLBati-bump-minor, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit c950e77 into OCA:14.0 Mar 23, 2023
6 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 2710561. Thanks a lot for contributing to OCA. ❤️

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.

None yet

4 participants