-
Notifications
You must be signed in to change notification settings - Fork 9.1k
[IMP] Frontdesk: update visitors dashboard #12978
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
Conversation
Hi @justinmallette - this is ready for a peer 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.
@larm-odoo LGTM! I have no comments on this one!
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.
Looks good, @larm-odoo !
a97af6a
to
df8a562
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.
Hi @larm-odoo! Nice work on this PR, approving with a quick fix on the images. Please address these before merge, thank you!
..
@robodoo delegate=larm-odoo
df8a562
to
3623046
Compare
@robodoo r+ |
New items appear in the visitors' dashboard, so updated instructions and images are needed. Updated formatting and verbiage here and there.