Skip to content

fix: apply user permission on attendance records (backport #4069)#4085

Merged
asmitahase merged 6 commits intoversion-16-hotfixfrom
mergify/bp/version-16-hotfix/pr-4069
Feb 5, 2026
Merged

fix: apply user permission on attendance records (backport #4069)#4085
asmitahase merged 6 commits intoversion-16-hotfixfrom
mergify/bp/version-16-hotfix/pr-4069

Conversation

@mergify
Copy link
Copy Markdown
Contributor

@mergify mergify bot commented Feb 5, 2026

Closes #3937, #3723

To-do

After

Screencast.From.2026-02-05.11-35-21.mp4

Summary by CodeRabbit

  • New Features

    • Added an option to include attendance records that lack check-in entries in the Shift Attendance report.
  • Bug Fixes

    • More complete reporting of shift attendance and improved filtering accuracy (including grace-period handling).
  • Tests

    • Added tests covering attendance-without-checkins, permission-based visibility, and chart-data validation.

This is an automatic backport of pull request #4069 done by [Mergify](https://mergify.com).

@mergify mergify bot requested a review from ruchamahabal as a code owner February 5, 2026 10:50
@mergify mergify bot requested a review from asmitahase as a code owner February 5, 2026 10:50
@asmitahase asmitahase merged commit 5001e61 into version-16-hotfix Feb 5, 2026
8 checks passed
@asmitahase asmitahase deleted the mergify/bp/version-16-hotfix/pr-4069 branch February 5, 2026 12:29
@frappe-pr-bot
Copy link
Copy Markdown
Collaborator

🎉 This PR is included in version 16.4.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants