-
Notifications
You must be signed in to change notification settings - Fork 737
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
feat(Desk + PWA): Geolocation in Employee Checkin (backport #1642) #1824
feat(Desk + PWA): Geolocation in Employee Checkin (backport #1642) #1824
Conversation
(cherry picked from commit e54e79b) # Conflicts: # hrms/hr/doctype/employee_checkin/employee_checkin.json
(cherry picked from commit 9023723) # Conflicts: # frontend/src/components/CheckInPanel.vue
(cherry picked from commit 66259a6)
(cherry picked from commit ad5407e)
(cherry picked from commit 72a3084)
(cherry picked from commit 74eeb3e) # Conflicts: # frontend/src/components/CheckInPanel.vue # hrms/api/__init__.py
…alidate (cherry picked from commit 239e1a4) # Conflicts: # frontend/src/components/CheckInPanel.vue # hrms/hr/doctype/employee_checkin/employee_checkin.json
(cherry picked from commit ced4969) # Conflicts: # hrms/hr/doctype/employee_checkin/employee_checkin.json
(cherry picked from commit 63e3b70)
(cherry picked from commit 8773889)
Cherry-pick of e54e79b has failed:
Cherry-pick of 9023723 has failed:
Cherry-pick of 74eeb3e has failed:
Cherry-pick of 239e1a4 has failed:
Cherry-pick of ced4969 has failed:
To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally |
Unrelated flaky test failure |
🎉 This PR is included in version 15.22.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
HR Settings
Employee Checkin (Desk)
geolocation-desk.mp4
Employee Checkin (Mobile App)
RPReplay_Final1712117350.MP4
Errors handled
Also if the setting is disabled, geolocation fields are hidden
This is an automatic backport of pull request #1642 done by [Mergify](https://mergify.com).