Skip to content

Show inactivity warning only after half a year - #491

Merged
rodlukas merged 1 commit into
masterfrom
feat-2026-10-29
Oct 29, 2026
Merged

rodlukas merged 1 commit into
masterfrom
feat-2026-10-29

Conversation

@rodlukas

@rodlukas rodlukas commented Oct 29, 2026 •

Copy link
Copy Markdown
Owner

Summary

  • The warning that an active client or group has had no lecture for too long now appears after 180 days (half a year) instead of 60 days.
  • The UI wording no longer mentions a number of days: "před více než půl rokem" in the tooltip and card warning, "déle než půl roku" in the banner above the Clients and Groups lists. The threshold can change again without the texts drifting.

Scope

Frontend only, a single threshold (DAYS_WITHOUT_LECTURE_WARNING) drives the badge on the lists, the warning on the card and the banner. No test or e2e step depended on the old value.

Testing

tsc --noEmit passes. Not checked in the browser.

@rodlukas rodlukas changed the title inactivity 180 days Show inactivity warning only after half a year Oct 29, 2026
@sonarqubecloud

Copy link
Copy Markdown

@rodlukas
rodlukas enabled auto-merge (squash) October 29, 2026 08:23
@codecov

codecov Bot commented Oct 29, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.06%. Comparing base (33ff17e) to head (5e0e9fe).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #491   +/-   ##
=======================================
  Coverage   97.06%   97.06%           
=======================================
  Files          22       22           
  Lines        1566     1566           
=======================================
  Hits         1520     1520           
  Misses         46       46           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@rodlukas
rodlukas merged commit 0e04dfb into master Oct 29, 2026
15 checks passed
@rodlukas
rodlukas deleted the feat-2026-10-29 branch October 29, 2026 08:30

This branch was successfully deployed

1 active deployment
testing — 5e0e9fee Deployed Oct 29, 2026 by rodlukas via Deploy to test Fly.io #1090
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.

1 participant