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][REF][l10n_br_account_nfe] lint #3269

Merged
merged 1 commit into from
Aug 8, 2024
Merged

[14.0][REF][l10n_br_account_nfe] lint #3269

merged 1 commit into from
Aug 8, 2024

Conversation

rvalyi
Copy link
Member

@rvalyi rvalyi commented Aug 7, 2024 •
edited
Loading

backport do lint da migraçao para a v15

algumas explicaçoes:

  • tirei um cls.invoice_demo_data.action_post() porque senao o erro era que o move ja tinha sido postado no teste e nao que tinha que prencher um meio fiscal de pagamento. Agora testamos esse UserError mesmo.
  • mudei um .post() por action_post() ja que post eh deprecated that v14

@OCA-git-bot
Copy link
Contributor

Hi @felipemotter, @antoniospneto, @mbcosta,
some modules you are maintaining are being modified, check this out!

@rvalyi rvalyi force-pushed the 14.0-account-nfe-lint branch 3 times, most recently from 1c7752f to 5559a81 Compare August 7, 2024 13:38
@rvalyi rvalyi mentioned this pull request Aug 7, 2024
Merged
@rvalyi
Copy link
Member Author

rvalyi commented Aug 8, 2024

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 14.0-ocabot-merge-pr-3269-by-rvalyi-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit f81db8a into OCA:14.0 Aug 8, 2024
7 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at ba64961. 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
Reviewers

@antoniospneto antoniospneto antoniospneto approved these changes

@marcelsavegnago marcelsavegnago marcelsavegnago approved these changes

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

Successfully merging this pull request may close these issues.
4 participants
@rvalyi @OCA-git-bot @antoniospneto @marcelsavegnago