Fix rendering of AbsenceReasonGroupSelect if no reasons are defined #189

Merged
hansegucker merged 4 commits from bugfix/fail-when-empty-reasons into master 2026-04-30 21:01:07 +02:00
Member

Information and screenshots

No further information have been provided.

Checklist

  • tox -e reformat,lint used
  • Changelog updated/No update necessary
  • Documentation (incl. Screenshots) up-to-date
  • Copyright in apps.py and README.rst up-to-date
  • (Unit) tests for new feature/regression tests for bug fixes (only if useful)
  • Adhere to URL scheme
  • Tested with admin account
  • Tested with normal account (based on permissions)

Backend checklist

  • Use permissions for all GraphQL mutations
  • Use query optimizer for all GraphQL queries
  • Don't use inefficient DB queries

Frontend checklist (mandatory with ~"part::frontend")

## Information and screenshots No further information have been provided. ## Checklist - [x] `tox -e reformat,lint` used - [ ] Changelog updated/No update necessary - [x] Documentation (incl. Screenshots) up-to-date - [ ] Copyright in `apps.py` and `README.rst` up-to-date - [x] (Unit) tests for new feature/regression tests for bug fixes (only if useful) - [x] Adhere to [URL scheme](https://edugit.org/AlekSIS/official/AlekSIS/-/issues/86) - [x] Tested with admin account - [x] Tested with normal account (based on permissions) ## Backend checklist - [ ] Use permissions for all GraphQL mutations - [ ] Use query optimizer for all GraphQL queries - [ ] Don't use inefficient DB queries ## Frontend checklist (mandatory with ~"part::frontend") - [ ] Screenshots are attached (mobile and desktop) - [ ] Tested on mobile, tablet, and desktop sizes - [x] Adhered to [frontend guidelines](https://edugit.org/AlekSIS/official/AlekSIS/-/blob/master/project/frontend_guidelines.rst?ref_type=heads) - [ ] Checked with [Firefox Accessibility Checker](https://firefox-source-docs.mozilla.org/devtools-user/accessibility_inspector/)
Author
Member

marked the checklist item tox -e reformat,lint used as completed

marked the checklist item **tox \-e reformat,lint used** as completed
Author
Member

marked the checklist item Documentation incl\. Screenshots up-to-date as completed

marked the checklist item **Documentation \(incl\. Screenshots\) up\-to\-date** as completed
Author
Member

marked the checklist item Unit tests for new feature/regression tests for bug fixes (only if useful) as completed

marked the checklist item **\(Unit\) tests for new feature/regression tests for bug fixes \(only if useful\)** as completed
Author
Member

marked the checklist item Adhere to URL scheme as completed

marked the checklist item **Adhere to URL scheme** as completed
Author
Member

marked the checklist item Tested with admin account as completed

marked the checklist item **Tested with admin account** as completed
Author
Member

marked the checklist item Tested with normal account (based on permissions) as completed

marked the checklist item **Tested with normal account \(based on permissions\)** as completed
Author
Member

marked the checklist item Adhered to frontend guidelines as completed

marked the checklist item **Adhered to frontend guidelines** as completed
Owner

@ZugBahnHof Please update the changelog.

@ZugBahnHof Please update the changelog.
Owner

assigned to @hansegucker

assigned to @hansegucker
Author
Member

added 1 commit

Compare with previous version

added 1 commit <ul><li>151f832f - Write Changelog</li></ul> [Compare with previous version](/AlekSIS/official/AlekSIS-App-Kolego/-/merge_requests/126/diffs?diff_id=95207&start_sha=775ede4fd9bd312ad6236aa19ccd2c79bd742073)
Author
Member

requested review from @hansegucker

requested review from @hansegucker
Author
Member

added 1 commit

Compare with previous version

added 1 commit <ul><li>733436d0 - Fix casing</li></ul> [Compare with previous version](/AlekSIS/official/AlekSIS-App-Kolego/-/merge_requests/126/diffs?diff_id=95211&start_sha=151f832fdfc8c8cb5e1b71c0753199a1720a103f)
Owner

added 3 commits

  • 733436d0...9586ad6e - 2 commits from branch master
  • fe84c56c - Merge branch 'master' into 'bugfix/fail-when-empty-reasons'

Compare with previous version

added 3 commits <ul><li>733436d0...9586ad6e - 2 commits from branch <code>master</code></li><li>fe84c56c - Merge branch &#39;master&#39; into &#39;bugfix/fail-when-empty-reasons&#39;</li></ul> [Compare with previous version](/AlekSIS/official/AlekSIS-App-Kolego/-/merge_requests/126/diffs?diff_id=95219&start_sha=733436d00d274b7f24f76f1cc75c5ea809612e5d)
Owner

mentioned in commit bddfc4b696

mentioned in commit bddfc4b6961cbc87080806b2f0c9231f6ddfb626
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
aleksis/AlekSIS-App-Kolego!189
No description provided.