project icon indicating copy to clipboard operation
project copied to clipboard

[16.0][ADD] project_task_name_with_id

Open AungKoKoLin1997 opened this issue 2 years ago • 7 comments

This module introduces improvements to our project task management. Now, users can locate tasks using not only the task name, but also the unique task ID. Furthermore, to make tasks easily distinguishable, each task's display name will now include its unique ID.

@qrtl

AungKoKoLin1997 avatar Jul 24 '23 05:07 AungKoKoLin1997

The usage overlaps with that of project_task_code https://github.com/OCA/project/pull/1084, but this module is simpler for those who do not need a specific sequence rule.

yostashiro avatar Jul 27 '23 01:07 yostashiro

@AungKoKoLin1997 could you add an USAGE file?

francesco-ooops avatar Sep 06 '23 13:09 francesco-ooops

could you add an USAGE file?

Done!

AungKoKoLin1997 avatar Sep 07 '23 02:09 AungKoKoLin1997

I believe we already have a module for this need: project_task_code. Would you consider migrating it to 16.0?

dreispt avatar Oct 10 '23 15:10 dreispt

@dreispt Please see this comment: https://github.com/OCA/project/pull/1158#issuecomment-1652799761

One reason I prefer this module over project_task_code is that id shows as the reference number in portal, therefore using id naturally makes the design more coherent with standard Odoo, IMO.

yostashiro avatar Oct 11 '23 02:10 yostashiro

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

OCA-git-bot avatar Jan 10 '24 08:01 OCA-git-bot

@dreispt Please see this comment: #1158 (comment)

One reason I prefer this module over project_task_code is that id shows as the reference number in portal, therefore using id naturally makes the design more coherent with standard Odoo, IMO.

@yostashiro thank you for a hint! I think one of the things TODO in the project_task_code module will be to redefine portal URLs using code instead of id.

ivs-cetmix avatar Jan 16 '24 21:01 ivs-cetmix

/ocabot merge nobump

dreispt avatar Apr 16 '24 08:04 dreispt

On my way to merge this fine PR! Prepared branch 16.0-ocabot-merge-pr-1158-by-dreispt-bump-nobump, awaiting test results.

OCA-git-bot avatar Apr 16 '24 08:04 OCA-git-bot

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

OCA-git-bot avatar Apr 16 '24 09:04 OCA-git-bot

@AungKoKoLin1997 Can you please check if you can make project_task_code and project_task_name_with_id coexist without conflicts by making parts of the functions configurable (or just by tweaking tests)? I will have to seggregate these modules from each other otherwise.

yostashiro avatar Apr 16 '24 10:04 yostashiro

@dreispt Can you please try to merge again?

AungKoKoLin1997 avatar Apr 17 '24 06:04 AungKoKoLin1997

/ocabot merge nobump

dreispt avatar Apr 17 '24 07:04 dreispt

What a great day to merge this nice PR. Let's do it! Prepared branch 16.0-ocabot-merge-pr-1158-by-dreispt-bump-nobump, awaiting test results.

OCA-git-bot avatar Apr 17 '24 07:04 OCA-git-bot

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

OCA-git-bot avatar Apr 17 '24 07:04 OCA-git-bot