-
Notifications
You must be signed in to change notification settings - Fork 8.4k
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
[IMP] Helpdesk: Update SLAs #11691
[IMP] Helpdesk: Update SLAs #11691
Conversation
hey @dikd-odoo this is ready for review. Thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @jero-odoo great work on this documentation update! I've finished my review with some edits that require your attention. Particularly I was wondering about the |SLA| command and how it builds out as "SLAs" only (plural), which is not always grammatically correct, depending on the context. Once you've taken care of that and looked at the other suggestions, feel free to move this to final review. Thank you!
7cac893
to
1484533
Compare
Thanks Dina! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @jero-odoo
Amazing job on this comprehensive doc! Your explanations are so clear, and your deep understanding of how the app is meant to be used really shines through—it’s a testament to your experience! 😊
I added a few minor rewording suggestions to help clarify the context for users who might not be as familiar with support teams and Helpdesk. Feel free to disregard them if they don’t fit or aren’t helpful. Keep up the fantastic work! 🙌
1484533
to
c99cd69
Compare
Thank you Felicia! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @jero-odoo, nice work on this PR 👍 Approving with two quick comments!
..
@robodoo delegate=jero-odoo
content/applications/services/helpdesk/overview/sla/sla-report-cohort.png
Outdated
Show resolved
Hide resolved
Co-authored-by: dikd-odoo <[email protected]>
c99cd69
to
dd2201b
Compare
@robodoo r+ |
closes #11691 Signed-off-by: Jessica Rogers (jero) <[email protected]> Co-authored-by: dikd-odoo <[email protected]>
closes #11691 Signed-off-by: Jessica Rogers (jero) <[email protected]> Co-authored-by: dikd-odoo <[email protected]>
closes #11691 Signed-off-by: Jessica Rogers (jero) <[email protected]> Co-authored-by: dikd-odoo <[email protected]>
@jero-odoo staging failed: ci/l10n on c28db857c04d717bc7afef265405af1fd814078a (view more at https://runbot.odoo.com/runbot/build/73137330) |
@robodoo retry |
Updated SLAs doc for V18. Updated images and content.
(Note: there is currently a bug in the bar graph that causes incorrect column headings to appear. Until that bug is worked out, those images have not been updated)