Skip to content

[IMP] expenses: reinvoicing #15699

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
Open
meval1006 wants to merge 1 commit into
base: 18.0
Choose a base branch
from
Open

[IMP] expenses: reinvoicing #15699

meval1006 wants to merge 1 commit into from

Conversation

@meval1006
Copy link
Contributor

@meval1006 meval1006 commented Dec 12, 2025

  • Updated screenshot to match GUI button changes
  • Removed expense category instructions and linked to the expense category page instead.

This 18.0 PR can be FWP up to master.

@meval1006 meval1006 requested a review from larm-odoo December 12, 2025 23:02
@meval1006 meval1006 self-assigned this Dec 12, 2025
@meval1006 meval1006 added the 2 label Dec 12, 2025
@robodoo
Copy link
Collaborator

robodoo commented Dec 12, 2025

Pull request status dashboard

@C3POdoo C3POdoo requested a review from a team December 12, 2025 23:04
larm-odoo
larm-odoo approved these changes Dec 16, 2025
View reviewed changes
Copy link
Contributor

@larm-odoo larm-odoo left a comment

Choose a reason for hiding this comment

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

Hi @meval1006 - great updates! I just had a bunch of suggestions for you - nothing major or necessary. Take what you want, ignore the rest =). There were only a few small grammatical things to change, but you'll know them when you read them. Approving now since I think you have this!

@meval1006 meval1006 force-pushed the 18.0-expenses-reinvoice-meval branch from e062fc7 to ba9f0de Compare December 16, 2025 21:40
@meval1006 meval1006 requested a review from Felicious December 16, 2025 21:41
@meval1006
Copy link
Contributor Author

meval1006 commented Dec 16, 2025

@Felicious Lara reviewed and approved this. The PR is now ready for final review.

Felicious
Felicious approved these changes Dec 17, 2025
View reviewed changes
Copy link
Contributor

@Felicious Felicious left a comment

Choose a reason for hiding this comment

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

Great work here, @meval1006 !

Just found some minor formatting and extra spaces and this is ready to merge!

@robodoo delegate+

When merging, use the command (intentional misspelling of robodoo so i don't accidentally trigger the command) @robotoo fw=no to merge but no fwport!

@meval1006 meval1006 force-pushed the 18.0-expenses-reinvoice-meval branch from 795a520 to 2927021 Compare December 17, 2025 19:03
@meval1006
Copy link
Contributor Author

meval1006 commented Dec 17, 2025

@robodoo fw=no

@robodoo
Copy link
Collaborator

robodoo commented Dec 17, 2025

Disabled forward-porting.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Felicious Felicious Felicious approved these changes

@larm-odoo larm-odoo larm-odoo approved these changes

Assignees

@meval1006 meval1006

Labels

2

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@meval1006 @robodoo @Felicious @larm-odoo