-
Notifications
You must be signed in to change notification settings - Fork 9.1k
[IMP] Attendances: updating kiosks #13004
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
base: 18.0
Are you sure you want to change the base?
Conversation
Hi @slinkous - this is ready for a peer review. |
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.
These are great updates. There were a couple of things missed by the linter (spelling/formatting) and I added a comments for a couple spots that tripped up my comprehension, which you can do with what you will!
60c546a
to
964d3ab
Compare
Hi @Felicious - this is ready for a final review =) |
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.
Thanks for your work on this, @larm-odoo ! 😄
One quick note on points—since 3-point PRs are meant for more substantial updates (typically around 50–80% of the doc’s length in new or revised content. This doc has ~180 lines and the lines added was 52), this one looks closer to a 2-point scope. Would you mind updating the label to reflect that? Thank you!
964d3ab
to
96e7946
Compare
Hi @samueljlieber - this is ready for a tech review! |
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.
Looks great to me @larm-odoo! Thank you :)
..
@robodoo r+
closes #13004 Signed-off-by: Samuel Lieber (sali) <[email protected]>
@larm-odoo @samueljlieber staging failed: ci/runbot on 4b5cf75138f59c22f38f5bf7bd2c21678b75d68c (view more at https://runbot.odoo.com/runbot/build/79109545) |
Updating to new formatting standards, new instructions, and new screenshots for the updated kiosk interface.