Skip to content

[18.0][MIG] product_alias (was shopinvader_product_alias) - #1206

Open
sebalix wants to merge 8 commits into
OCA:18.0from
akretion:18.0-mig-shopinvader_product_alias
Open

[18.0][MIG] product_alias (was shopinvader_product_alias)#1206
sebalix wants to merge 8 commits into
OCA:18.0from
akretion:18.0-mig-shopinvader_product_alias

Conversation

@sebalix

@sebalix sebalix commented May 11, 2026

Copy link
Copy Markdown

Migrate shopinvader_product_alias (which was hosted in a PR on shopinvader/odoo-shopinvader) to 18.0 and rename it product_alias as it doesn't depend at all on Shopinvader.

@sebalix
sebalix force-pushed the 18.0-mig-shopinvader_product_alias branch from b6b0431 to ecc80e5 Compare May 11, 2026 13:23
@sebalix
sebalix marked this pull request as ready for review May 11, 2026 13:33

@paradoxxxzero paradoxxxzero left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM (code & functional review)

Comment thread product_alias/readme/DESCRIPTION.md Outdated
@@ -0,0 +1,2 @@
This module allows to add url aliases from the product template for each

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

URL? I see no URL feature here 🤔

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Also, how is this different from having product links of type "alias"?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Comment thread product_alias/models/product_product.py Outdated
Comment thread product_alias/models/product_alias.py
Comment thread product_alias/__manifest__.py Outdated
Comment thread product_alias/models/product_alias.py Outdated
Comment thread product_alias/models/product_alias.py

@metaminux metaminux left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Only one typo in DESCRIPTION.md, but LGTM

Comment thread product_alias/models/product_alias.py
Comment thread product_alias/models/product_alias.py
Comment thread product_alias/readme/DESCRIPTION.md Outdated
@OCA-git-bot

Copy link
Copy Markdown
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). 🤖

@sebalix
sebalix force-pushed the 18.0-mig-shopinvader_product_alias branch from 4916ac7 to 7d2c27c Compare August 4, 2026 06:28
@metaminux

Copy link
Copy Markdown

Hello @simahawk

Can you review that PR again ?

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.

7 participants