Skip to content
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.

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

[15.0][IMP] helpdesk_mgmt: Add specific use cases with team without company_id #620

Merged
merged 1 commit into from
Sep 10, 2024

Conversation

victoralmau
Copy link
Member

@victoralmau victoralmau commented Sep 5, 2024

Add specific use cases with team without company_id

Related to #553 (comment)

Please @pedrobaeza and @pilarvargas-tecnativa can you review it?

@Tecnativa

@victoralmau
Copy link
Member Author

victoralmau commented Sep 5, 2024

It is not possible to set default_team_id because that will cause stage_id to be set, and currently users are expected to set the team_id field manually @eLBati

@pedrobaeza

This comment was marked as duplicate.

@victoralmau victoralmau force-pushed the 15.0-imp-helpdesk_mgmt-misc-company branch from 46ef9a9 to 0e6d569 Compare September 5, 2024 16:02
@pedrobaeza pedrobaeza requested a review from eLBati September 5, 2024 16:11
@OCA-git-bot
Copy link
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). 🤖

@pedrobaeza
Copy link
Member

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 15.0-ocabot-merge-pr-620-by-pedrobaeza-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 4330654 into OCA:15.0 Sep 10, 2024
7 checks passed
@OCA-git-bot
Copy link
Contributor

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

@pedrobaeza pedrobaeza deleted the 15.0-imp-helpdesk_mgmt-misc-company branch September 10, 2024 12:12
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.

4 participants