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

[17.0][MIG] sale_layout_category_hide_detail: Migration to 17.0 #253

Merged

Conversation

jeroenev
Copy link

No description provided.

ernestotejeda and others added 30 commits February 21, 2024 16:56
Hide details for sections in sales orders and invoices for reports and
customer portal
Currently translated at 100.0% (7 of 7 strings)

Translation: sale-reporting-11.0/sale-reporting-11.0-sale_layout_category_hide_detail
Translate-URL: https://translation.odoo-community.org/projects/sale-reporting-11-0/sale-reporting-11-0-sale_layout_category_hide_detail/es/
…nvoice"

Propagate values in 'Show details' and 'Show subtotal' fields from Sale Order Lines to Invoice Lines
… tests + linting + JS for column not in fieldsInfo list
If a note line is put in first position, we get an error due to this missing condition.

[UPD] Update sale_layout_category_hide_detail.pot

[UPD] README.rst

Update translation files

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: sale-reporting-13.0/sale-reporting-13.0-sale_layout_category_hide_detail
Translate-URL: https://translation.odoo-community.org/projects/sale-reporting-13-0/sale-reporting-13-0-sale_layout_category_hide_detail/

[UPD] README.rst
Currently translated at 100.0% (11 of 11 strings)

Translation: sale-reporting-13.0/sale-reporting-13.0-sale_layout_category_hide_detail
Translate-URL: https://translation.odoo-community.org/projects/sale-reporting-13-0/sale-reporting-13-0-sale_layout_category_hide_detail/es/
Add a third clickable option on sections to hide all amounts in the
detailed lines of a section.

[UPD] Update sale_layout_category_hide_detail.pot

sale_layout_category_hide_detail 13.0.1.1.0

Update translation files

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: sale-reporting-13.0/sale-reporting-13.0-sale_layout_category_hide_detail
Translate-URL: https://translation.odoo-community.org/projects/sale-reporting-13-0/sale-reporting-13-0-sale_layout_category_hide_detail/
…shed invoices or confirmed orders.

Related to: OCA#128 (comment)

TT38579
…olumns of the buttons so that the table takes up as little space as possible.

TT38579
…o be sure boolean_fa_icon work fine in all use cases

TT38579
…on subtotal' button details and add images to visualize each buttons effects
Change the fa icon Widget to new owl Component
Copy link

@navarrorico navarrorico left a comment

Choose a reason for hiding this comment

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

Tested on runboat. The icon buttons of the section are not working.

@jeroenev
Copy link
Author

jeroenev commented Mar 4, 2024

Did you test on a Quote or confirmed SO? buttons are only clickable when the quote is not yet confirmed.
This was the same in Odoo16, but I know this was not the case in Odoo14, which might cause some confusion if you know the module from an older Odoo version.
Work correctly on this quote: http://oca-sale-reporting-17-0-pr253-e9ff0b161700.runboat.odoo-community.org/web#id=3&cids=1&menu_id=183&action=313&model=sale.order&view_type=form

Copy link

@navarrorico navarrorico left a comment

Choose a reason for hiding this comment

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

@jeroenev True!!! Sorry. All wroks fine then. Thank you

Functional review LGTM 👍

@pedrobaeza pedrobaeza changed the title 17.0 mig sale layout category hide detail [17.0][MIG] sale_layout_category_hide_detail: Migration to 17.0 Mar 4, 2024
@pedrobaeza
Copy link
Member

/ocabot migration sale_layout_category_hide_detail

@OCA-git-bot OCA-git-bot added this to the 17.0 milestone Mar 4, 2024
@willievmobile
Copy link

@jeroenev ive really tried for weeks i cannot get this to work on a odoo17 thats been updated to the version starting this month..?
Im desperately needing this to work for the company so we can use odoo17 as without this we have to make 2 quotes for every job. please help?

@Ricardoalso
Copy link

Ricardoalso commented Jun 11, 2024

@jeroenev ive really tried for weeks i cannot get this to work on a odoo17 thats been updated to the version starting this month..? Im desperately needing this to work for the company so we can use odoo17 as without this we have to make 2 quotes for every job. please help?

Hello @willievmobile, can you give me more context please ?

@willievmobile
Copy link

@jeroenev I gave up on this issue and just installed odoo 16 and we are working on that.
But basically it looked like it was how the server from Cloudclusters was installed or something. but i just dont know enough to figure it out.

@trisdoan
Copy link

@jeroenev I gave up on this issue and just installed odoo 16 and we are working on that. But basically it looked like it was how the server from Cloudclusters was installed or something. but i just dont know enough to figure it out.

Hello @willievmobile, I guess the issue you encountered is 500 Internal Server Error

If yes, I prepared an PR to fix it in jeroenev#1. Hello @jeroenev, could you plz take this fix into account?

@jeroenev jeroenev force-pushed the 17.0-mig-sale_layout_category_hide_detail branch from d695328 to d58f989 Compare June 13, 2024 13:26
@jeroenev
Copy link
Author

jeroenev commented Jun 13, 2024

@trisdoan any instructions how I would integrate that PR into mine? Not too well versed in the OCA workflows for that

@trisdoan
Copy link

Hello @jeroenev, you can go to my PR and merge like this

sample

@jeroenev
Copy link
Author

OK that was easier than expected, thanks a lot!

@victoralmau
Copy link
Member

Please, cherry-pick #276 to commit history (before migration commit)

@OCA-git-bot
Copy link
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@arianzaghi
Copy link

@jeroenev Hi! How long is left before the merge?

@yvaucher
Copy link
Member

yvaucher commented Aug 8, 2024

As the PR is ready, we can tackle the FW port of #276 in a later PR.

@yvaucher
Copy link
Member

yvaucher commented Aug 8, 2024

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 17.0-ocabot-merge-pr-253-by-yvaucher-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit fc730b2 into OCA:17.0 Aug 8, 2024
6 of 7 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 339dfec. 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
Projects
None yet
Development

Successfully merging this pull request may close these issues.