Skip to content

[19.0][MIG] project_tag_security: Migration to 19.0#1712

Open
stferraro wants to merge 12 commits into
OCA:19.0from
Jenrax-git:19.0-mig-project_tag_security
Open

[19.0][MIG] project_tag_security: Migration to 19.0#1712
stferraro wants to merge 12 commits into
OCA:19.0from
Jenrax-git:19.0-mig-project_tag_security

Conversation

@stferraro
Copy link
Copy Markdown

No description provided.

@OCA-git-bot OCA-git-bot added series:19.0 mod:project_tag_security Module project_tag_security labels Apr 23, 2026
@stferraro stferraro mentioned this pull request Apr 23, 2026
42 tasks
Copy link
Copy Markdown

@WesleyOliveira98 WesleyOliveira98 left a comment

Choose a reason for hiding this comment

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

@stferraro stferraro force-pushed the 19.0-mig-project_tag_security branch from 29302a2 to c937a29 Compare April 26, 2026 02:41
Copy link
Copy Markdown

@WesleyOliveira98 WesleyOliveira98 left a comment

Choose a reason for hiding this comment

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

The PR title stll not following the OCA patterns, should be:
[19.0][MIG] <module>: Migration to 19.0

https://github.com/OCA/maintainer-tools/wiki/Migration-to-version-19.0#how-to
Image

@stferraro stferraro force-pushed the 19.0-mig-project_tag_security branch from c937a29 to bcdd5f7 Compare April 28, 2026 01:00
Copy link
Copy Markdown

@WesleyOliveira98 WesleyOliveira98 left a comment

Choose a reason for hiding this comment

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

@stferraro you need to change the PR title to [19.0][MIG] <module>: Migration to 19.0, the commit title was correct, should be [MIG] <module>: Migration to 19.0

Image

Everything else is ok, just change this

@stferraro stferraro force-pushed the 19.0-mig-project_tag_security branch from bcdd5f7 to 4671fd7 Compare April 28, 2026 12:52
Copy link
Copy Markdown

@WesleyOliveira98 WesleyOliveira98 left a comment

Choose a reason for hiding this comment

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

  • In your PR, click on this button:
Image
  • Insert the title: [19.0][MIG] project_tag_security: Migration to 19.0

  • Click on save

Image
  • Then, change the commit message to: [MIG] project_tag_security: Migration to 19.0

@stferraro stferraro changed the title 19.0 mig project tag security [19.0][MIG] project_tag_security: Migration to 19.0 Apr 28, 2026
@stferraro stferraro force-pushed the 19.0-mig-project_tag_security branch from 4671fd7 to c30977d Compare April 28, 2026 19:53
@stferraro
Copy link
Copy Markdown
Author

@CristianoMafraJunior please check this!

Comment thread project_tag_security/__manifest__.py Outdated
# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl).
{
"name": "Project Tag Security",
"version": "19.0.1.0.1",
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

19.0.1.0.0

@stferraro stferraro force-pushed the 19.0-mig-project_tag_security branch from c30977d to b3c3ef5 Compare April 28, 2026 20:10
@stferraro stferraro force-pushed the 19.0-mig-project_tag_security branch from b3c3ef5 to 8cfdb61 Compare April 28, 2026 20:13
Copy link
Copy Markdown

@WesleyOliveira98 WesleyOliveira98 left a comment

Choose a reason for hiding this comment

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

LGTM

@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). 🤖

Copy link
Copy Markdown

@luisDIXMIT luisDIXMIT left a comment

Choose a reason for hiding this comment

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

Code review and tested on runboat, LGTM!

@leemannd
Copy link
Copy Markdown
Contributor

leemannd commented May 4, 2026

/ocabot migration project_tag_security

@OCA-git-bot OCA-git-bot added this to the 19.0 milestone May 4, 2026
Copy link
Copy Markdown

@Yusuke1998 Yusuke1998 left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Copy Markdown

@Karla-Villegas Karla-Villegas left a comment

Choose a reason for hiding this comment

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

Functional tests: Approved.

Copy link
Copy Markdown
Contributor

@BhaveshHeliconia BhaveshHeliconia left a comment

Choose a reason for hiding this comment

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

LGTM!

@stferraro
Copy link
Copy Markdown
Author

@OCA-git-bot ping

@leemannd
Copy link
Copy Markdown
Contributor

@OCA-git-bot ping

Hello @stferraro I don't know this. I'm not sure it does anything

@leemannd
Copy link
Copy Markdown
Contributor

/ocabot merge nobump

@OCA-git-bot
Copy link
Copy Markdown
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 19.0-ocabot-merge-pr-1712-by-leemannd-bump-nobump, awaiting test results.

@OCA-git-bot
Copy link
Copy Markdown
Contributor

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

@stferraro
Copy link
Copy Markdown
Author

stferraro commented May 21, 2026

/ocabot merge nobump

@leemannd
Copy link
Copy Markdown
Contributor

There is an error when trying to merge it

2026-05-21 13:27:29,168 335 INFO odoo odoo.addons.project_task_description_template.tests.test_project_task: Starting TestDescriptionTemplate.test_onchange_description_template_id ... 
2026-05-21 13:27:29,193 335 INFO odoo odoo.addons.project_task_description_template.tests.test_project_task: ====================================================================== 
2026-05-21 13:27:29,193 335 ERROR odoo odoo.addons.project_task_description_template.tests.test_project_task: FAIL: TestDescriptionTemplate.test_onchange_description_template_id
Traceback (most recent call last):
  File "/__w/project/project/project_task_description_template/tests/test_project_task.py", line 22, in test_onchange_description_template_id
    self.assertEqual(
AssertionError: Markup('<p>Existing Description</p><span>- Sample Description</span>') != Markup('<p>Existing Description</p><p>- Sample Description</p>') : Onchange method failed to append description correctly.
 
2026-05-21 13:27:29,194 335 INFO odoo odoo.addons.project_task_description_template.tests.test_project_task: Starting TestDescriptionTemplate.test_onchange_with_empty_description ... 
2026-05-21 13:27:29,198 335 INFO odoo odoo.addons.project_task_description_template.tests.test_project_task: ====================================================================== 
2026-05-21 13:27:29,198 335 ERROR odoo odoo.addons.project_task_description_template.tests.test_project_task: FAIL: TestDescriptionTemplate.test_onchange_with_empty_description
Traceback (most recent call last):
  File "/__w/project/project/project_task_description_template/tests/test_project_task.py", line 32, in test_onchange_with_empty_description
    self.assertEqual(
AssertionError: Markup('<span>- Sample Description</span>') != Markup('<p>- Sample Description</p>') : Onchange method failed with empty initial description.```

@leemannd
Copy link
Copy Markdown
Contributor

@stferraro do you have time to have look at it?

@stferraro
Copy link
Copy Markdown
Author

@leemannd
Captura de pantalla 2026-05-21 a las 10 40 05
Suggested Fix:
Please update the expected assertion value from 18 to 19 in both test methods (around lines 119 and 224). Also, there is a small typo in line 107: "A Should have A subtasks" which could be cleaned up to "A Should have 9 subtasks".
Thanks for your work on this! in test!

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.