8000 [ADD] Appraisals: Appraisal templates by larm-odoo · Pull Request #13244 · odoo/documentation · GitHub
[go: up one dir, main page]
Skip to content

[ADD] Appraisals: Appraisal templates #13244

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

[ADD] Appraisals: Appraisal templates #13244

wants to merge 1 commit into from

Conversation

larm-odoo
Copy link
Contributor

Adding new doc about templates.

@larm-odoo larm-odoo self-assigned this May 7, 2025
@larm-odoo larm-odoo added the 5 label May 7, 2025
@robodoo
Copy link
Collaborator
robodoo commented May 7, 2025

Pull request status dashboard

@larm-odoo larm-odoo requested a review from a team May 7, 2025 18:15
@C3POdoo C3POdoo requested a review from a team May 7, 2025 18:18
@larm-odoo
Copy link
Contributor Author

Hi @Felicious - this is ready for a peer review!

Felicious
Felicious approved these changes May 9, 2025
View reviewed changes
Copy link
Contributor
@Felicious Felicious left a comment

Choose a reason for hiding this comment

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

LGTM @larm-odoo !

Felicious
Felicious approved these changes May 9, 2025
View reviewed changes
Copy link
Contributor
@Felicious Felicious left a comment

Choose a reason for hiding this comment

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

Came back to nitpick the grammar in the template 😅 feel free to ignore if you'd like to remain faithful to the template

@larm-odoo larm-odoo force-pushed the 18.0-appraisals-appraisal-templates-larm branch from 715a3d3 to 5241093 Compare May 9, 2025 20:01
@larm-odoo larm-odoo requested a review from a team May 9, 2025 20:02
@larm-odoo
Copy link
Contributor Author

Hi @samueljlieber - this is ready for a tech review!

@larm-odoo larm-odoo force-pushed the 18.0-appraisals-appraisal-templates-larm branch from 5241093 to 406f626 Compare May 12, 2025 14:09
samueljlieber
samueljlieber approved these changes May 12, 2025
View reviewed changes
Copy link
Contributor
@samueljlieber samueljlieber left a comment

Choose a reason for hiding this comment

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

Hi @larm-odoo, thank you for your work on this PR!

I am approving with a comment about the table, up to your discretion. Please compress the image before merge, thank you! :)
.....
@robodoo delegate=larm-odoo

@larm-odoo larm-odoo force-pushed the 18.0-appraisals-appraisal-templates-larm branch from 406f626 to 8476349 Compare May 12, 2025 17:11
@larm-odoo
Copy link
Contributor Author

@robodoo r+

@robodoo robodoo closed this in cbbee0b May 12, 2025
This was referenced May 12, 2025
Closed
Closed
Closed
@Felicious Felicious mentioned this pull request May 28, 2025
Closed
This was referenced May 30, 2025
Closed
Closed
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Felicious Felicious Felicious approved these changes

@samueljlieber samueljlieber samueljlieber approved these changes

Assignees

@larm-odoo larm-odoo

Labels
5
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@larm-odoo @robodoo @Felicious @samueljlieber
0