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] rating: Use with_company() in the rating controller #159100

Open
wants to merge 1 commit into
base: 15.0
Choose a base branch
from

Conversation

victoralmau
Copy link
Contributor

If we use multi-company and create a rating related to a record that has a company_id set, that company must be used for the header to be correct (e.g. company logo).

Example use case:

  • Create Company A and set a logo
  • Create Company B and set a logo
  • Creates a project linked to Company B
  • Create a New and Done stage linked to the previous project.
  • Go to Project > Configuration and activate Customer Ratings
  • Set a rating template (Task: Rating Request) to the Done stage.
  • Create a task linked to Company B and a customer.
  • Change task to the Done stage.
  • Click on the satisfaction icon in the email sent to the customer.

Current behavior before PR

  • The correct logo (company B) is displayed.

@Tecnativa TT48498


I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

@robodoo
Copy link
Contributor

robodoo commented Mar 25, 2024

Pull request status dashboard.

@C3POdoo C3POdoo requested a review from a team March 25, 2024 13:06
If we use multi-company and create a rating related to a record that has a company_id set,
that company must be used for the header to be correct (e.g. company logo).

Example use case:
- Create Company A and set a logo
- Create Company B and set a logo
- Creates a project linked to Company B
- Create a New and Done stage linked to the previous project.
- Go to Project > Configuration and activate Customer Ratings
- Set a rating template (Task: Rating Request) to the Done stage.
- Create a task linked to Company B and a customer.
- Change task to the Done stage.
- Click on the satisfaction icon in the email sent to the customer.

TT48498
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants