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.

Sign up for GitHub

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

Jump to bottom

[14.0] [FIX] l10n_it_fatturapa_out: Fix error tests to copy files #2303

Merged
merged 1 commit into from
Sep 17, 2021
Merged

[14.0] [FIX] l10n_it_fatturapa_out: Fix error tests to copy files #2303

merged 1 commit into from
Sep 17, 2021

Conversation

TheMule71
Copy link
Contributor

Forward-port of #2051

--
Confermo di aver firmato il CLA https://odoo-community.org/page/cla e di aver letto le linee guida su https://odoo-community.org/page/contributing

@TheMule71 TheMule71 requested a review from eLBati May 28, 2021 20:00
@eLBati eLBati changed the title [FIX] l10n_it_fatturapa_out: Fix error tests to copy files [14.0] [FIX] l10n_it_fatturapa_out: Fix error tests to copy files Jun 1, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jun 18, 2021
@TheMule71
merged OCA#1989 OCA#1990 OCA#2128 - added OCA#2303 OCA#2305 OCA#2309 O…
363028b 
…CA#2311 OCA#2314 OCA#2335 OCA#2336 OCA#2310

PR OCA#2151 under investigation
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jun 18, 2021
@TheMule71
merged OCA#1989 OCA#1990 OCA#2128 - added OCA#2303 OCA#2305 OCA#2309 O…
4a17685 
…CA#2311 OCA#2314 OCA#2335 OCA#2336 OCA#2310

PR OCA#2151 under investigation
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jun 18, 2021
@TheMule71
merged OCA#1989 OCA#1990 OCA#2128 - added OCA#2303 OCA#2305 OCA#2309 O…
35dc26b 
…CA#2311 OCA#2314 OCA#2335 OCA#2336 OCA#2310

PR OCA#2151 under investigation
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 2, 2021
@TheMule71
merged OCA#1989 OCA#1990 OCA#2128 - added OCA#2303 OCA#2305 OCA#2309 O…
002cf0c 
…CA#2311 OCA#2314 OCA#2335 OCA#2336 OCA#2310

PR OCA#2151 under investigation
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 2, 2021
@TheMule71
merged OCA#1989 OCA#1990 OCA#2128 - added OCA#2303 OCA#2305 OCA#2309 O…
3cb7d78 
…CA#2311 OCA#2314 OCA#2335 OCA#2336 OCA#2310

PR OCA#2151 under investigation
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 9, 2021
@TheMule71
merged OCA#1989 OCA#1990 OCA#2128 - added OCA#2303 OCA#2305 OCA#2309 O…
0b6cd29 
…CA#2311 OCA#2314 OCA#2335 OCA#2336 OCA#2310

PR OCA#2151 under investigation
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 9, 2021
@TheMule71
merged OCA#1989 OCA#1990 OCA#2128 - added OCA#2303 OCA#2305 OCA#2309 O…
fdad49a 
…CA#2311 OCA#2314 OCA#2335 OCA#2336 OCA#2310

PR OCA#2151 under investigation
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 9, 2021
@TheMule71
merged OCA#1989 OCA#1990 OCA#2128 - added OCA#2303 OCA#2305 OCA#2309 O…
385e9b8 
…CA#2311 OCA#2314 OCA#2335 OCA#2336 OCA#2310

PR OCA#2151 under investigation
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 9, 2021
@TheMule71
merged OCA#1989 OCA#1990 OCA#2128 - added OCA#2303 OCA#2305 OCA#2309 O…
c5401de 
…CA#2311 OCA#2314 OCA#2335 OCA#2336 OCA#2310

PR OCA#2151 under investigation
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 17, 2021
@TheMule71
merged OCA#1989 OCA#1990 OCA#2128 - added OCA#2303 OCA#2305 OCA#2309 O…
4e01fb6 
…CA#2311 OCA#2314 OCA#2335 OCA#2336 OCA#2310

PR OCA#2151 under investigation
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 20, 2021
@TheMule71
merged OCA#1989 OCA#1990 OCA#2128 - added OCA#2303 OCA#2305 OCA#2309 O…
cb72666 
…CA#2311 OCA#2314 OCA#2335 OCA#2336 OCA#2310

PR OCA#2151 under investigation
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 30, 2021
@TheMule71
merged OCA#1989 OCA#1990 OCA#2128 - added OCA#2303 OCA#2305 OCA#2309 O…
a5d604f 
…CA#2311 OCA#2314 OCA#2335 OCA#2336 OCA#2310

PR OCA#2151 under investigation
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 30, 2021
@TheMule71
merged OCA#1989 OCA#1990 OCA#2128 - added OCA#2303 OCA#2305 OCA#2309 O…
eefd01c 
…CA#2311 OCA#2314 OCA#2335 OCA#2336 OCA#2310

PR OCA#2151 under investigation
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Aug 6, 2021
@TheMule71
merged OCA#1989 OCA#1990 OCA#2128 - added OCA#2303 OCA#2305 OCA#2309 O…
4c1e787 
…CA#2311 OCA#2314 OCA#2335 OCA#2336 OCA#2310

PR OCA#2151 under investigation
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Aug 20, 2021
@TheMule71
merged OCA#1989 OCA#1990 OCA#2128 - added OCA#2303 OCA#2305 OCA#2309 O…
e8874ce 
…CA#2311 OCA#2314 OCA#2335 OCA#2336 OCA#2310

PR OCA#2151 under investigation
SimoRubi
SimoRubi approved these changes Aug 20, 2021
View reviewed changes
Copy link
Member

@SimoRubi SimoRubi left a comment

Choose a reason for hiding this comment

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

Revisione del codice, per me è ok.
Non la vedo come una cosa bloccante, ma riusciresti a mantenere l'autore originale del commit?
Attualmente il commit risulta tuo:
image
ma penso sarebbe più corretta una cosa tipo:
image
come succede per le migrazioni

TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Sep 3, 2021
@TheMule71
merged OCA#1989 OCA#1990 OCA#2128 - added OCA#2303 OCA#2305 OCA#2309 O…
178aacb 
…CA#2311 OCA#2314 OCA#2335 OCA#2336 OCA#2310

PR OCA#2151 under investigation
@TheMule71 @victoralmau
[FIX] l10n_it_fatturapa_out: Fix error tests to copy files
7d625be 
Co-Authored-By: Víctor Martínez <victor.martinez@tecnativa.com>
@TheMule71
Copy link
Contributor Author

Revisione del codice, per me è ok.
Non la vedo come una cosa bloccante, ma riusciresti a mantenere l'autore originale del commit?
Attualmente il commit risulta tuo:
image
ma penso sarebbe più corretta una cosa tipo:
image
come succede per le migrazioni

Grazie, di solito lo faccio, stavolta mi è scappato.

@TheMule71
Copy link
Contributor Author

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 14.0-ocabot-merge-pr-2303-by-TheMule71-bump-patch, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Sep 17, 2021
@OCA-git-bot
Merge PR #2303 into 14.0
43c1767 
Signed-off-by TheMule71
@OCA-git-bot
Copy link
Contributor

It looks like something changed on 14.0 in the meantime.
Let me try again (no action is required from you).
Prepared branch 14.0-ocabot-merge-pr-2303-by-TheMule71-bump-patch, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Sep 17, 2021
@OCA-git-bot
Merge PR #2303 into 14.0
b8993d1 
Signed-off-by TheMule71
@OCA-git-bot
Copy link
Contributor

It looks like something changed on 14.0 in the meantime.
Let me try again (no action is required from you).
Prepared branch 14.0-ocabot-merge-pr-2303-by-TheMule71-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit f48e6f7 into OCA:14.0 Sep 17, 2021
@OCA-git-bot
Copy link
Contributor

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

TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Sep 17, 2021
@TheMule71
merged OCA#1989 OCA#1990 OCA#2128 - added OCA#2303 OCA#2305 OCA#2309 O…
8e6053b 
…CA#2311 OCA#2314 OCA#2335 OCA#2336 OCA#2310

PR OCA#2151 under investigation
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Oct 1, 2021
@TheMule71
merged OCA#1989 OCA#1990 OCA#2128 - added OCA#2303 OCA#2305 OCA#2309 O…
0d5a27f 
…CA#2311 OCA#2314 OCA#2335 OCA#2336 OCA#2310

PR OCA#2151 under investigation
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Oct 1, 2021
@TheMule71
merged OCA#2043 2080# OCA#2303 OCA#2305 OCA#2358 OCA#2419 OCA#2427, a…
2b7460e 
…dded OCA#2432 OCA#2442 OCA#2446, norebase OCA#2360
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Oct 8, 2021
@TheMule71
merged OCA#2043 2080# OCA#2303 OCA#2305 OCA#2358 OCA#2419 OCA#2427, a…
faaf55f 
…dded OCA#2432 OCA#2442 OCA#2446, norebase OCA#2360
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Oct 8, 2021
@TheMule71
merged OCA#1989 OCA#1990 OCA#2128 - added OCA#2303 OCA#2305 OCA#2309 O…
d45ba17 
…CA#2311 OCA#2314 OCA#2335 OCA#2336 OCA#2310

PR OCA#2151 under investigation
@TheMule71 TheMule71 deleted the 14.0-forward-2051 branch October 15, 2021 16:19
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Oct 15, 2021
@TheMule71
merged OCA#1989 OCA#1990 OCA#2128 - added OCA#2303 OCA#2305 OCA#2309 O…
c1e3192 
…CA#2311 OCA#2314 OCA#2335 OCA#2336 OCA#2310

PR OCA#2151 under investigation
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Oct 15, 2021
@TheMule71
merged OCA#2043 2080# OCA#2303 OCA#2305 OCA#2358 OCA#2419 OCA#2427, a…
1ca71e7 
…dded OCA#2432 OCA#2442 OCA#2446, norebase OCA#2360
TheMule71 added a commit to TheMule71/l10n-italy that referenced this pull request Jan 21, 2022
@TheMule71
merged OCA#1989 OCA#1990 OCA#2128 - added OCA#2303 OCA#2305 OCA#2309 O…
39d7b3f 
…CA#2311 OCA#2314 OCA#2335 OCA#2336 OCA#2310

PR OCA#2151 under investigation
TheMule71 added a commit to TheMule71/l10n-italy that referenced this pull request Jan 21, 2022
@TheMule71
merged OCA#2043 2080# OCA#2303 OCA#2305 OCA#2358 OCA#2419 OCA#2427, a…
8e2131d 
…dded OCA#2432 OCA#2442 OCA#2446, norebase OCA#2360
TheMule71 added a commit to TheMule71/l10n-italy that referenced this pull request Jan 21, 2022
@TheMule71
merged OCA#1989 OCA#1990 OCA#2128 - added OCA#2303 OCA#2305 OCA#2309 O…
f247096 
…CA#2311 OCA#2314 OCA#2335 OCA#2336 OCA#2310

PR OCA#2151 under investigation
TheMule71 added a commit to TheMule71/l10n-italy that referenced this pull request Jan 21, 2022
@TheMule71
merged OCA#2043 2080# OCA#2303 OCA#2305 OCA#2358 OCA#2419 OCA#2427, a…
21dddf7 
…dded OCA#2432 OCA#2442 OCA#2446, norebase OCA#2360
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jun 29, 2022
@TheMule71
merged OCA#1989 OCA#1990 OCA#2128 - added OCA#2303 OCA#2305 OCA#2309 O…
0f96484 
…CA#2311 OCA#2314 OCA#2335 OCA#2336 OCA#2310

PR OCA#2151 under investigation
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jun 29, 2022
@TheMule71
merged OCA#2043 2080# OCA#2303 OCA#2305 OCA#2358 OCA#2419 OCA#2427, a…
132eef5 
…dded OCA#2432 OCA#2442 OCA#2446, norebase OCA#2360
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@SimoRubi SimoRubi SimoRubi approved these changes

@eLBati eLBati Awaiting requested review from eLBati

Assignees
No one assigned
Labels
merged 🎉
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@TheMule71 @OCA-git-bot @SimoRubi