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

[IMP] Validate payment information #1457

Closed
wants to merge 1 commit into from

Conversation

mileo
Copy link
Member

@mileo mileo commented Jun 12, 2021

No description provided.

@mileo mileo changed the title [FIX] Validate payment information [IMP] Validate payment information Jun 12, 2021
@mileo
Copy link
Member Author

mileo commented Jul 20, 2021

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 12.0-ocabot-merge-pr-1457-by-mileo-bump-nobump, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Jul 20, 2021
Signed-off-by mileo
@OCA-git-bot
Copy link
Contributor

@mileo your merge command was aborted due to failed check(s), which you can inspect on this commit of 12.0-ocabot-merge-pr-1457-by-mileo-bump-nobump.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

@rvalyi
Copy link
Member

rvalyi commented Jul 21, 2021

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 12.0-ocabot-merge-pr-1457-by-rvalyi-bump-nobump, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Jul 21, 2021
Signed-off-by rvalyi
@OCA-git-bot
Copy link
Contributor

@rvalyi your merge command was aborted due to failed check(s), which you can inspect on this commit of 12.0-ocabot-merge-pr-1457-by-rvalyi-bump-nobump.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

@rvalyi
Copy link
Member

rvalyi commented Jul 21, 2021

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 12.0-ocabot-merge-pr-1457-by-rvalyi-bump-nobump, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Jul 21, 2021
Signed-off-by rvalyi
@OCA-git-bot
Copy link
Contributor

@rvalyi your merge command was aborted due to failed check(s), which you can inspect on this commit of 12.0-ocabot-merge-pr-1457-by-rvalyi-bump-nobump.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

@rvalyi
Copy link
Member

rvalyi commented Jul 21, 2021

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 12.0-ocabot-merge-pr-1457-by-rvalyi-bump-nobump, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Jul 21, 2021
Signed-off-by rvalyi
@OCA-git-bot
Copy link
Contributor

@rvalyi your merge command was aborted due to failed check(s), which you can inspect on this commit of 12.0-ocabot-merge-pr-1457-by-rvalyi-bump-nobump.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

@rvalyi
Copy link
Member

rvalyi commented Jul 21, 2021

o @mbcosta e o @renatonlima acham que esse PR vai solucionar o erro que esta acontecendo de forma aleatoria nos testes (de acordo com a ordem de instação dos modulos): #1523

@rvalyi
Copy link
Member

rvalyi commented Jul 21, 2021

ola @rvalyi o erro que está acontecendo aqui pela mensagem se trata de uma validação do pre-commit https://github.com/OCA/l10n-brazil/runs/3124320728 https://app.travis-ci.com/github/OCA/l10n-brazil/jobs/525962898
image

puts...
Eh que o Travis tinha passado antes da gente ativa o pre-commit no l10n_br_nfe por isso pensei que era o outro erro aleatorio sem checar direito...

@rvalyi
Copy link
Member

rvalyi commented Jul 26, 2021

superseded by #1538 with proper pre-commit formatting.

@rvalyi rvalyi closed this Jul 26, 2021
rvalyi added a commit to akretion/l10n-brazil that referenced this pull request Aug 12, 2021
rvalyi added a commit to akretion/l10n-brazil that referenced this pull request Oct 18, 2021
rvalyi added a commit to akretion/l10n-brazil that referenced this pull request Oct 18, 2021
rvalyi added a commit to akretion/l10n-brazil that referenced this pull request Oct 18, 2021
rvalyi added a commit to akretion/l10n-brazil that referenced this pull request Oct 18, 2021
rvalyi added a commit to akretion/l10n-brazil that referenced this pull request Oct 18, 2021
rvalyi added a commit to akretion/l10n-brazil that referenced this pull request Oct 19, 2021
rvalyi added a commit to akretion/l10n-brazil that referenced this pull request Nov 8, 2021
renatonlima pushed a commit to akretion/l10n-brazil that referenced this pull request Nov 10, 2021
rvalyi added a commit to akretion/l10n-brazil that referenced this pull request Jan 4, 2022
renatonlima pushed a commit to akretion/l10n-brazil that referenced this pull request Jan 6, 2022
renatonlima pushed a commit to akretion/l10n-brazil that referenced this pull request Feb 5, 2022
rvalyi added a commit to akretion/l10n-brazil that referenced this pull request Jun 10, 2022
rvalyi added a commit to akretion/l10n-brazil that referenced this pull request Jan 17, 2024
rvalyi added a commit to akretion/l10n-brazil that referenced this pull request May 22, 2024
rvalyi added a commit to akretion/l10n-brazil that referenced this pull request May 23, 2024
rvalyi added a commit to akretion/l10n-brazil that referenced this pull request Jul 20, 2024
rvalyi added a commit to akretion/l10n-brazil that referenced this pull request Jul 21, 2024
rvalyi added a commit to akretion/l10n-brazil that referenced this pull request Jul 23, 2024
rvalyi added a commit to akretion/l10n-brazil that referenced this pull request Jul 23, 2024
rvalyi added a commit to akretion/l10n-brazil that referenced this pull request Jul 23, 2024
rvalyi added a commit to akretion/l10n-brazil that referenced this pull request Jul 24, 2024
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.

5 participants