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] crm_partner_assign: move fields before user_id #512

Merged
merged 1 commit into from
Aug 2, 2023

Conversation

EmilioPascual
Copy link
Contributor

@EmilioPascual EmilioPascual commented Aug 1, 2023

Fields partner contact assigned and partner assigned aren't related to customer. They shouldn't be before phone or email in views.

@moduon MT-3175 @rafaelbn @Shide

Opportunities:

image

Leads:

image

- Fields partner contact assigned and partner assigned aren't related customer like phone or email. They shouldn't be before phone or email in views.

@moduon MT-3175 @rafaelbn @Shide

Co-authored-by: Rafael Blasco <[email protected]>
@OCA-git-bot
Copy link
Contributor

Hi @Shide,
some modules you are maintaining are being modified, check this out!

@EmilioPascual EmilioPascual changed the title [15.0][IMP] crm_project_assign: move fields before user_id [15.0][IMP] crm_partner_assign: move fields before user_id Aug 1, 2023
@rafaelbn rafaelbn added this to the 15.0 milestone Aug 1, 2023
Copy link
Member

@rafaelbn rafaelbn left a comment

Choose a reason for hiding this comment

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

Tested in runbot 👍🏼

Copy link

@fcvalgar fcvalgar left a comment

Choose a reason for hiding this comment

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

Functional review.

LGTM, thank you @EmilioPascual

@rafaelbn
Copy link
Member

rafaelbn commented Aug 2, 2023

/ocabot merge minor

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 15.0-ocabot-merge-pr-512-by-rafaelbn-bump-minor, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 6c65584 into OCA:15.0 Aug 2, 2023
6 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 99486f9. 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