Skip to content

Conversation

Arubaru86
Copy link

This module ensures that geolocation is required for employee check-ins and check-outs
in the Odoo attendance system. Employees must have GPS enabled and allow location
permissions in their browser before they can clock in or out.

Features:

  • Prevents attendance if geolocation permissions are denied.
  • Displays a warning message when GPS is not enabled.
  • Ensures that all attendances include latitude and longitude.
  • Works in Kiosk Mode and Manual Mode.
  • Extends public_kiosk_app.js to enforce geolocation checks.

Impact:

  • Guarantees location-based attendance tracking.
  • Improves data accuracy for remote workers.
  • Ensures compliance with location-based attendance policies.

@Arubaru86 Arubaru86 force-pushed the 17.0-hr_attendance_geolocation_required branch from 9a5b871 to f7a0a70 Compare March 14, 2025 14:06
Copy link
Member

@Tardo Tardo left a comment

Choose a reason for hiding this comment

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

Please, add tests... and the 'readme' folder with "CONTRIBUTORS, DESCRIPTION, ..." files.

Copy link
Member

@victoralmau victoralmau left a comment

Choose a reason for hiding this comment

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

I think another one could be to add this directly in hr_attendance_geolocation through a boolean field at res.config.settings level and activate it who needs it. What do you think?

@pedrobaeza
Copy link
Member

AFAIK, hr_attendance_geolocation is no longer needed in 17, as it's included in core. Can you confirm?

@victoralmau
Copy link
Member

victoralmau commented Mar 14, 2025

AFAIK, hr_attendance_geolocation is no longer needed in 17, as it's included in core. Can you confirm?

You're right, also I had it noted internally (I didn't remember it in the quick comment above) but it looks like OpenUpgrade is missing, I'll add it soon to avoid confusion.

OCA/OpenUpgrade#4693

@Arubaru86 Arubaru86 force-pushed the 17.0-hr_attendance_geolocation_required branch from 1e63c20 to 96bdbb8 Compare June 9, 2025 10:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants