+
Skip to content

Conversation

ap-wtioit
Copy link
Contributor

While reviewing for 19.0 #3297 i discovered that:

The description inside Odoo's Module Info featured a broken image:
image

With this change it is fixed:
image

If this is ok, i will forward port to 18.0 and 19.0 (17.0 seems to be the first version with the broken description)

Info @wt-io-it

@ap-wtioit ap-wtioit changed the title [FIX] web_environment_ribbon: fix configuration image in description [17.0][FIX] web_environment_ribbon: fix configuration image in description Oct 1, 2025
@pedrobaeza pedrobaeza added this to the 17.0 milestone Oct 3, 2025
{
"name": "Web Environment Ribbon",
"version": "17.0.1.0.2",
"version": "17.0.1.0.3",
Copy link
Member

Choose a reason for hiding this comment

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

Please don't change module version manually.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Well, if it happens automatic, which place would it increase? I fear that the automatic one would increase it to 17.0.1.1.0 while this is only a fix without functional changes so it feels like the last place needs to be incremented. Do you know were i can lookup the automatic OCA version increment code?

Copy link
Member

Choose a reason for hiding this comment

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

It depends on how we (PSCs/core maintainers/maintainers) launch the command. We have ocabot merge patch/minor/major.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks for the explanation, i removed the change of the version number from my commit.

@ap-wtioit ap-wtioit force-pushed the 17.0-fix_web_environment_ribbon_description branch from a5cff3a to e36e83c Compare October 7, 2025 06:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

点击 这是indexloc提供的php浏览器服务,不要输入任何密码和下载