Skip to content

feat: WFH days custom field + daily leave/WFH notifications#3

Merged
NagariaHussain merged 1 commit into
developfrom
claude/github-issue-fix-ra5850
Jun 25, 2026
Merged

feat: WFH days custom field + daily leave/WFH notifications#3
NagariaHussain merged 1 commit into
developfrom
claude/github-issue-fix-ra5850

Conversation

@NagariaHussain

Copy link
Copy Markdown
Contributor

Issue #1 (Half Day WFH):

  • Add custom field custom_total_wfh_days (Float) on Attendance Request
    via after_install/after_migrate hooks.
  • WFH handler now captures total WFH days in this field (0.5 for half day,
    N for an N-day request) instead of ticking the Half Day checkbox, so the
    day is no longer split into half present / half absent in attendance.

Issue #2 (Leave and WFH Notifier):

  • Add daily scheduled tasks broadcasting to whitelisted chats:
    • 10:30 AM: employees on approved leave + employees working from home
    • 3:00 PM: second WFH reminder
  • Add broadcast_to_whitelisted_chats helper in telegram_utils.

Co-Authored-By: Claude Opus 4.8 noreply@anthropic.com
Claude-Session: https://claude.ai/code/session_01TUAVgnJYq6C3wZQCttUPfr

Issue #1 (Half Day WFH):
- Add custom field `custom_total_wfh_days` (Float) on Attendance Request
  via after_install/after_migrate hooks.
- WFH handler now captures total WFH days in this field (0.5 for half day,
  N for an N-day request) instead of ticking the Half Day checkbox, so the
  day is no longer split into half present / half absent in attendance.

Issue #2 (Leave and WFH Notifier):
- Add daily scheduled tasks broadcasting to whitelisted chats:
  - 10:30 AM: employees on approved leave + employees working from home
  - 3:00 PM: second WFH reminder
- Add broadcast_to_whitelisted_chats helper in telegram_utils.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01TUAVgnJYq6C3wZQCttUPfr
@NagariaHussain NagariaHussain merged commit 622c06a into develop Jun 25, 2026
1 of 3 checks passed
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.

2 participants