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

[17.0][MIG] helpdesk_mgmt_stage_server_action #650

Open
wants to merge 3 commits into
base: 17.0
Choose a base branch
from

Conversation

yankinmax
Copy link

No description provided.

@yankinmax yankinmax force-pushed the 17.0-mig-helpdesk_mgmt_stage_server_action branch from 28d6eec to 2dfcf38 Compare October 29, 2024 16:31
@yankinmax
Copy link
Author

@ivantodorovich I wasn't sure to understand the cases where we need to check the stage is changed, so I have simplified the code. Thanks for pointing to this, I'll update.

@yankinmax yankinmax force-pushed the 17.0-mig-helpdesk_mgmt_stage_server_action branch 2 times, most recently from 79f3e2e to e748030 Compare November 4, 2024 13:59
@ivantodorovich
Copy link

travis is red, could you check? 🙏🏻

@yankinmax yankinmax force-pushed the 17.0-mig-helpdesk_mgmt_stage_server_action branch from e748030 to 44f1fc6 Compare November 4, 2024 14:23
@yankinmax yankinmax force-pushed the 17.0-mig-helpdesk_mgmt_stage_server_action branch from 44f1fc6 to b35a1d9 Compare November 12, 2024 13:03
@ivantodorovich
Copy link

@yankinmax base PR is merged, could you remove the test dependency?

@yankinmax yankinmax force-pushed the 17.0-mig-helpdesk_mgmt_stage_server_action branch from b35a1d9 to 193597c Compare December 4, 2024 12:49
@yankinmax yankinmax force-pushed the 17.0-mig-helpdesk_mgmt_stage_server_action branch from a4f65d6 to 5809a6b Compare December 5, 2024 13:25
@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). 🤖

@gurneyalex
Copy link
Member

Can you check why we have both README.md and README.rst?

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.

6 participants