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

[14.0][ADD] helpdesk_mgmt_timesheet: add optional ticket field in view #477

Merged
merged 1 commit into from
Jul 10, 2023

Conversation

SergiCForgeFlow
Copy link
Contributor

This adds an optional ticket field in Project Task timesheets tree view.

Copy link
Contributor

@LoisRForgeFlow LoisRForgeFlow left a comment

Choose a reason for hiding this comment

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

Functional test, LGTM

Copy link
Contributor

@dalonsod dalonsod left a comment

Choose a reason for hiding this comment

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

See comment

@@ -261,4 +261,14 @@
</field>
</field>
</record>
<record id="view_task_form2_inherited" model="ir.ui.view">
Copy link
Contributor

Choose a reason for hiding this comment

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

This view should be placed in its own project.task xml file.

Also, not tested and not sure, this could lead to security issues, when user is not allowed to Helpdesk app. Could you check?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Checking!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Made the following changes:

  • moved the view to another xml file related with project.task.
  • added a group restriction to helpdesk user so that only users in that group can view the field.
    @dalonsod

@SergiCForgeFlow SergiCForgeFlow force-pushed the 14.0-add-helpdesk_mgmt_timesheet branch from 8fc0fbd to 88da8c6 Compare June 30, 2023 08:37
Copy link
Contributor

@dalonsod dalonsod left a comment

Choose a reason for hiding this comment

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

👍

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

@LoisRForgeFlow
Copy link
Contributor

@pedrobaeza good to merge?

@pedrobaeza pedrobaeza added this to the 14.0 milestone Jul 10, 2023
@SergiCForgeFlow SergiCForgeFlow force-pushed the 14.0-add-helpdesk_mgmt_timesheet branch from 88da8c6 to fa5010a Compare July 10, 2023 14:24
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.

/ocabot merge minor

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 14.0-ocabot-merge-pr-477-by-pedrobaeza-bump-minor, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 3ac6625 into OCA:14.0 Jul 10, 2023
4 checks passed
@OCA-git-bot
Copy link
Contributor

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

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.

5 participants