[17.0][OU-ADD] website_sale: Nothing to do

[pilarvargas-tecnativa]

cc @Tecnativa TT51563

[legalsylvain]

/ocabot migration website_sale

[hbrunn]

I think this PR should include merging website_sale_digital into website_sale and run the query @dansanti proposes in #4561 (comment)

[pilarvargas-tecnativa]

I think this PR should include merging website_sale_digital into website_sale and run the query @dansanti proposes in #4561 (comment)

Hello @hbrunn ,

Thank you for your suggestion. It is preferable to make a different PR for each module. The website_sale_digital script will be done later. Although it's always welcome if you prefer to do it yourself to get ahead of the work.

Regards

[dansanti]

hi! @pilarvargas-tecnativa website_sale_digital was merged with website_sale please add this scripts:

openupgrade.logged_query(
    env.cr,
    """
    UPDATE product_document
    SET attached_on = 'sale_order'
    FROM ir_attachment ir
    WHERE product_document.ir_attachment_id = ir.id
      AND ir.product_downloadable = TRUE
    """
)```

[pilarvargas-tecnativa]

hi! @pilarvargas-tecnativa website_sale_digital was merged with website_sale please add this scripts:

Hi @dansanti thanks for your work. I had already commented it with @hbrunn before but I reiterate my words, it's better to make a separate PR for each module. I take into account your suggestions to do it later but it's always welcome if someone wants to do that PR.

My regards

[MiquelRForgeFlow]

/ocabot merge nobump

[OCA-git-bot]

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 17.0-ocabot-merge-pr-4687-by-MiquelRForgeFlow-bump-nobump, awaiting test results.

[OCA-git-bot]

@MiquelRForgeFlow your merge command was aborted due to failed check(s), which you can inspect on this commit of 17.0-ocabot-merge-pr-4687-by-MiquelRForgeFlow-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.

[MiquelRForgeFlow]

/ocabot merge nobump

[OCA-git-bot]

On my way to merge this fine PR!
Prepared branch 17.0-ocabot-merge-pr-4687-by-MiquelRForgeFlow-bump-nobump, awaiting test results.

[OCA-git-bot]

Congratulations, your PR was merged at 741f443. Thanks a lot for contributing to OCA. ❤️

[MiquelRForgeFlow]

@pilarvargas-tecnativa please do website_sale_digital :)

[pedrobaeza]

We don't need that module for now.

[MiquelRForgeFlow]

website_sale_digital handled in #4879.