Skip to content

feat: Implement Geolocation-Based Smart Attendance Verification (#3917)#3951

Open
knoxiboy wants to merge 1 commit into
Premshaw23:masterfrom
knoxiboy:3917-feat-geolocation-attendance
Open

feat: Implement Geolocation-Based Smart Attendance Verification (#3917)#3951
knoxiboy wants to merge 1 commit into
Premshaw23:masterfrom
knoxiboy:3917-feat-geolocation-attendance

Conversation

@knoxiboy

@knoxiboy knoxiboy commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Is your feature request related to a problem? Please describe.

Students can mark attendance from outside the classroom, which makes proxy attendance possible and weakens trust in attendance records. This issue needs a geolocation gate so attendance can only be validated near the configured classroom location. (Resolves #3917)

Describe the solution you'd like

I added geolocation support to the attendance settings flow and normalized the backend schema so classroom location data can be stored and returned consistently. The attendance validation flow now uses a shared useGeolocation helper for current-position checks and distance calculations, keeping the client-side verification aligned with the backend settings payload.

Describe alternatives you've considered

An alternative was to keep the geolocation logic entirely inside the attendance component, but extracting the shared helper makes the location behavior easier to reuse and keeps the validation flow easier to maintain.

Additional Context

The PR keeps the existing passcode-based validation intact and only adds the geolocation verification path plus the related settings persistence needed for the feature.

Copilot AI review requested due to automatic review settings July 2, 2026 18:08

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

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.

[Feature] Implement Geolocation-Based Smart Attendance Verification

2 participants