Skip to content

[16.0][FIX] helpdesk_mgmt: remove redundant duplicate action #697

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
aleuffre wants to merge 1 commit into
base: 16.0
Choose a base branch
from
Open

[16.0][FIX] helpdesk_mgmt: remove redundant duplicate action #697

aleuffre wants to merge 1 commit into from

Conversation

@aleuffre
Copy link
Contributor

@aleuffre aleuffre commented Feb 18, 2025 •
edited
Loading

The same result can be achieved by installing the OCA module web_tree_duplicate

(Duplicate duplicate, heh)

@aleuffre aleuffre force-pushed the 16.0-helpdesk-ticket-duplicate branch from af0ce75 to 4bbcf29 Compare February 18, 2025 11:28
@anusriNPS
Copy link
Contributor

anusriNPS commented Feb 18, 2025

Code Review LGTM

@OCA-git-bot
Copy link
Contributor

OCA-git-bot commented Feb 24, 2025

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

@flotho
Copy link
Member

flotho commented Jul 22, 2025

hi @aleuffre , any chance to rebase for CI/CD ?

@aleuffre
[FIX] helpdesk_mgmt: remove redundant duplicate action
deae218 
The same result can be achieved by installing the OCA module
`web_tree_duplicate`
@aleuffre aleuffre force-pushed the 16.0-helpdesk-ticket-duplicate branch from 4bbcf29 to deae218 Compare July 22, 2025 12:05
@aleuffre
Copy link
Contributor Author

aleuffre commented Jul 22, 2025

hi @aleuffre , any chance to rebase for CI/CD ?

Done!

ivantodorovich
ivantodorovich approved these changes Aug 12, 2025
View reviewed changes
Copy link
Contributor

@ivantodorovich ivantodorovich left a comment

Choose a reason for hiding this comment

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

Agree, this is redundant

@pedrobaeza pedrobaeza added this to the 17.0 milestone Aug 12, 2025
pedrobaeza
pedrobaeza requested changes Aug 12, 2025
View reviewed changes
Copy link
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

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

Requiring extra modules for such simple option for me is not the same, as it increases the overall maintenance. I would agree to remove it on upper versions where you can duplicate more than one record at the same time natively.

@github-actions
Copy link

github-actions bot commented Dec 14, 2025

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Dec 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pedrobaeza pedrobaeza pedrobaeza requested changes

@marcelsavegnago marcelsavegnago marcelsavegnago approved these changes

@dalonsod dalonsod Awaiting requested review from dalonsod

+4 more reviewers

@ivantodorovich ivantodorovich ivantodorovich approved these changes

@christian-ramos-tecnativa christian-ramos-tecnativa christian-ramos-tecnativa approved these changes

@CristianoMafraJunior CristianoMafraJunior CristianoMafraJunior approved these changes

@anusriNPS anusriNPS anusriNPS approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

stale PR/Issue without recent activity, it'll be soon closed automatically.

Projects

None yet

Milestone

17.0

Development

Successfully merging this pull request may close these issues.

9 participants

@aleuffre @anusriNPS @OCA-git-bot @flotho @ivantodorovich @marcelsavegnago @pedrobaeza @christian-ramos-tecnativa @CristianoMafraJunior