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

[14.0][FIX] l10n_it_fatturapa_in: Fixed singleton error on field computation #2350

Closed
wants to merge 1 commit into from
Closed

Conversation

cloudition
Copy link

Fixes: #2349

Descrizione del problema o della funzionalità:

Comportamento attuale prima di questa PR:

Comportamento desiderato dopo questa PR:

--
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
Copy link
Contributor

Hi there!

Please run:
$ pre-commit run -a
review and add changed file(s)
$ git commit --amend
$ git push -f
if you want to get rid of the pre-commit failed check.

If you're not familiar with the above procedure (which could be destructive if executed in the wrong repo/branch), please feel free to ask here.

@cloudition
Copy link
Author

Hi,

I'm not an OCA member and I'm not familiar with your travis checks/deployment. I have fixed this issue locally for my customer and just wanted to report it here for you to know. The quick patch I just submited because I was asked to. I'm sorry but I have no more time to invest here... If you want you can reject/close the PR or take the code and submit it yourown :-)

@TheMule71
Copy link
Contributor

Hi,

I'm not an OCA member and I'm not familiar with your travis checks/deployment. I have fixed this issue locally for my customer and just wanted to report it here for you to know. The quick patch I just submited because I was asked to. I'm sorry but I have no more time to invest here... If you want you can reject/close the PR or take the code and submit it yourown :-)

No worries, thank you for your PR. Every contribution is more than welcome. :)

@TheMule71
Copy link
Contributor

Closing, please refer to #2355

@TheMule71 TheMule71 closed this Jul 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants