Resolve "Add vue-draggable-grid based UI for planning supervisions" #198

Merged
hansegucker merged 35 commits from 75-add-vue-draggable-grid-based-ui-for-planning-supervisions into master 2025-07-30 13:36:10 +02:00
Member

Closes #75

Information and screenshots

image.png

Screenshot From 2025-07-29 14-46-55.png

Screenshot From 2025-07-29 14-46-51.png

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")

  • Screenshots are attached (mobile and desktop)
  • Tested on mobile, tablet, and desktop sizes
  • Adhered to frontend guidelines
  • Checked with Firefox Accessibility Checker
Closes #75 ## Information and screenshots ![image.png](/uploads/fe47b34707d7be2146f668919f2bac1e/image.png) ![Screenshot From 2025-07-29 14-46-55.png](/uploads/780f21c7b654c256fc67b12ffee04fa6/Screenshot_From_2025-07-29_14-46-55.png) ![Screenshot From 2025-07-29 14-46-51.png](/uploads/3931342ad57e8ae95603773fdcca4dea/Screenshot_From_2025-07-29_14-46-51.png) ## Checklist - [x] `tox -e reformat,lint` used - [x] Changelog updated/No update necessary - [ ] Documentation (incl. Screenshots) up-to-date - [x] Copyright in `apps.py` and `README.rst` up-to-date - [ ] (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 - [x] Use permissions for all GraphQL mutations - [x] Use query optimizer for all GraphQL queries - [x] Don't use inefficient DB queries ## Frontend checklist (mandatory with ~"part::frontend") - [x] Screenshots are attached (mobile and desktop) - [x] Tested on mobile, tablet, and desktop sizes - [x] Adhered to frontend guidelines - [x] Checked with Firefox Accessibility Checker
Author
Member

assigned to @yuha

assigned to @yuha
Author
Member

added 1 commit

  • 39e99268 - WIP: First draft of supervision planning grid

Compare with previous version

added 1 commit <ul><li>39e99268 - WIP: First draft of supervision planning grid</li></ul> [Compare with previous version](/AlekSIS/official/AlekSIS-App-Lesrooster/-/merge_requests/110/diffs?diff_id=67310&start_sha=0a43e83977ce9fcb75648d5c8322be18094ae2eb)
Author
Member

added 2 commits

  • 3df10e97 - First draft of supervision planning grid
  • be7a4923 - Add proper text to delete dialog

Compare with previous version

added 2 commits <ul><li>3df10e97 - First draft of supervision planning grid</li><li>be7a4923 - Add proper text to delete dialog</li></ul> [Compare with previous version](/AlekSIS/official/AlekSIS-App-Lesrooster/-/merge_requests/110/diffs?diff_id=67342&start_sha=39e992688a1c8bdbdb42c602c2fb60e31d70acab)
Author
Member

added 2 commits

  • 1ca0c9e8 - Fix supervision edit form
  • ac2a9b3b - Add break icon to time period card

Compare with previous version

added 2 commits <ul><li>1ca0c9e8 - Fix supervision edit form</li><li>ac2a9b3b - Add break icon to time period card</li></ul> [Compare with previous version](/AlekSIS/official/AlekSIS-App-Lesrooster/-/merge_requests/110/diffs?diff_id=67343&start_sha=be7a49236db4435ee4211b6dacb18097787b0fc2)
Author
Member

added 6 commits

  • ec62b1eb - Use PersonChip instead of colored short name chip
  • 651e1f8d - Adapt seach filter function
  • 7ef08129 - Drop unused lesson ratio computed method
  • 3a556bc9 - Drop unused dragged rooms computed method
  • 2b2e03f4 - Implement blocking overlay
  • 13cee99c - Add supervisions mini timetable

Compare with previous version

added 6 commits <ul><li>ec62b1eb - Use PersonChip instead of colored short name chip</li><li>651e1f8d - Adapt seach filter function</li><li>7ef08129 - Drop unused lesson ratio computed method</li><li>3a556bc9 - Drop unused dragged rooms computed method</li><li>2b2e03f4 - Implement blocking overlay</li><li>13cee99c - Add supervisions mini timetable</li></ul> [Compare with previous version](/AlekSIS/official/AlekSIS-App-Lesrooster/-/merge_requests/110/diffs?diff_id=67564&start_sha=ac2a9b3b1cdc965468439b0aa4153f21bdcaf8fa)
Author
Member

added 1 commit

Compare with previous version

added 1 commit <ul><li>03cedb49 - Reformat</li></ul> [Compare with previous version](/AlekSIS/official/AlekSIS-App-Lesrooster/-/merge_requests/110/diffs?diff_id=67565&start_sha=13cee99ced091f3fe574d81010acf2e0cea2d775)
Author
Member

added 3 commits

  • e0ac730f - Also show rooms in mini supervision plan
  • 87aaf94b - Use correct translation keys
  • 2de420dd - Adapt to correct url schema

Compare with previous version

added 3 commits <ul><li>e0ac730f - Also show rooms in mini supervision plan</li><li>87aaf94b - Use correct translation keys</li><li>2de420dd - Adapt to correct url schema</li></ul> [Compare with previous version](/AlekSIS/official/AlekSIS-App-Lesrooster/-/merge_requests/110/diffs?diff_id=67575&start_sha=03cedb496173e7a764ac70fe3bc9db2a73d330e1)
Author
Member

added 1 commit

Compare with previous version

added 1 commit <ul><li>03fbc594 - Adapt to correct url schema</li></ul> [Compare with previous version](/AlekSIS/official/AlekSIS-App-Lesrooster/-/merge_requests/110/diffs?diff_id=67577&start_sha=2de420dd2867d51bea5eb8af05636608373d9bd6)
Author
Member

added 2 commits

Compare with previous version

added 2 commits <ul><li>3a2ad02a - Add todo message</li><li>b9c651a5 - Empty break slots on VR change</li></ul> [Compare with previous version](/AlekSIS/official/AlekSIS-App-Lesrooster/-/merge_requests/110/diffs?diff_id=67578&start_sha=03fbc5947bf07809b1b2e986442c1f465714b857)
Author
Member

added 2 commits

  • 98934965 - Use shorter button texts and add icons
  • 143aa3fa - Remove unused TeacherCard

Compare with previous version

added 2 commits <ul><li>98934965 - Use shorter button texts and add icons</li><li>143aa3fa - Remove unused TeacherCard</li></ul> [Compare with previous version](/AlekSIS/official/AlekSIS-App-Lesrooster/-/merge_requests/110/diffs?diff_id=67923&start_sha=b9c651a5f9cb0ee272ba9ab5faf2d9c53625b055)
Author
Member

added 6 commits

  • 345c8663 - Don't show teachers' full names in supervision card
  • adb88b49 - Show subject if present in supervision card
  • c463a180 - Remove unneeded section of code
  • 44a1a05c - Set new nudge value for supervision edit menu
  • 6c8e3e29 - Use non-breaking hyphen
  • aa9833a2 - Adapt permissions

Compare with previous version

added 6 commits <ul><li>345c8663 - Don&#39;t show teachers&#39; full names in supervision card</li><li>adb88b49 - Show subject if present in supervision card</li><li>c463a180 - Remove unneeded section of code</li><li>44a1a05c - Set new nudge value for supervision edit menu</li><li>6c8e3e29 - Use non-breaking hyphen</li><li>aa9833a2 - Adapt permissions</li></ul> [Compare with previous version](/AlekSIS/official/AlekSIS-App-Lesrooster/-/merge_requests/110/diffs?diff_id=67993&start_sha=143aa3fafae999b005cf2e4d8f4678c836249b17)
Author
Member

added 1 commit

Compare with previous version

added 1 commit <ul><li>27fcd598 - Reformat</li></ul> [Compare with previous version](/AlekSIS/official/AlekSIS-App-Lesrooster/-/merge_requests/110/diffs?diff_id=67996&start_sha=aa9833a25f3ef35508d77b5028cd4901a257e6c2)
Author
Member

added 1 commit

Compare with previous version

added 1 commit <ul><li>c0ce7d59 - Add changelog</li></ul> [Compare with previous version](/AlekSIS/official/AlekSIS-App-Lesrooster/-/merge_requests/110/diffs?diff_id=67998&start_sha=27fcd598ead31b6b53c877cedef763234ff39ab0)
Author
Member

marked this merge request as ready

marked this merge request as **ready**
Author
Member

changed the description

changed the description
Owner

changed the description

changed the description
Owner

Please check if they are all used.

Please check if they are all used.
Owner

Please check this.

Please check this.
Owner

Drop

Drop
Owner

Drop

Drop
Owner

Drop

Drop
Owner
      path: "supervision/",
```suggestion:-0+0 path: "supervision/", ```
Owner

Please use this specific global thing.

Please use this specific global thing.
Owner

requested changes

requested changes
Author
Member

changed this line in version 13 of the diff

changed this line in [version 13 of the diff](/AlekSIS/official/AlekSIS-App-Lesrooster/-/merge_requests/110/diffs?diff_id=68003&start_sha=c0ce7d59d393f513d819806deecb682e851037d7#30af248e98a10c2e99bfcf74af42a8cd253ba8ed_129_129)
Author
Member

added 1 commit

  • a6694051 - Apply 1 suggestion(s) to 1 file(s)

Compare with previous version

added 1 commit <ul><li>a6694051 - Apply 1 suggestion(s) to 1 file(s)</li></ul> [Compare with previous version](/AlekSIS/official/AlekSIS-App-Lesrooster/-/merge_requests/110/diffs?diff_id=68003&start_sha=c0ce7d59d393f513d819806deecb682e851037d7)
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 Changelog updated/No update necessary as completed

marked the checklist item **Changelog updated/No update necessary** 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 Documentation (incl. Screenshots) up-to-date as incomplete

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

added 2 commits

  • 5866f7eb - Update copyright
  • 520895f3 - Merge branch '75-add-vue-draggable-grid-based-ui-for-planning-supervisions' of...

Compare with previous version

added 2 commits <ul><li>5866f7eb - Update copyright</li><li>520895f3 - Merge branch &#39;75-add-vue-draggable-grid-based-ui-for-planning-supervisions&#39; of...</li></ul> [Compare with previous version](/AlekSIS/official/AlekSIS-App-Lesrooster/-/merge_requests/110/diffs?diff_id=68006&start_sha=a6694051fd520e1145927741a14d2351a51054d8)
Author
Member

marked the checklist item Copyright in apps.py and README.rst up-to-date as completed

marked the checklist item **Copyright in `apps.py` and `README.rst` up-to-date** as completed
Author
Member

marked the checklist item Adhere to URL scheme as completed

marked the checklist item **Adhere to [URL scheme](https://edugit.org/AlekSIS/official/AlekSIS/-/issues/86)** 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 Don't use inefficient DB queries as completed

marked the checklist item **Don't use inefficient DB queries** as completed
Author
Member

marked the checklist item Use query optimizer for all GraphQL queries as completed

marked the checklist item **Use query optimizer for all GraphQL queries** as completed
Author
Member

marked the checklist item Screenshots are attached (mobile and desktop) as completed

marked the checklist item **Screenshots are attached (mobile and desktop)** as completed
Author
Member

marked the checklist item Tested on mobile, tablet, and desktop sizes as completed

marked the checklist item **Tested on mobile, tablet, and desktop sizes** as completed
Author
Member

marked the checklist item Checked with Firefox Accessibility Checker as completed

marked the checklist item **Checked with Firefox Accessibility Checker** as completed
Author
Member

marked the checklist item Adhered to frontend guidelines as completed

marked the checklist item **Adhered to frontend guidelines** as completed
Author
Member

resolved all threads

resolved all threads
Author
Member

changed this line in version 15 of the diff

changed this line in [version 15 of the diff](/AlekSIS/official/AlekSIS-App-Lesrooster/-/merge_requests/110/diffs?diff_id=68396&start_sha=520895f3847754716b3068208565e0ffe7cdafe3#c43f57521338b8464b72e54e5b89ae888d1510d4_56_56)
Author
Member

changed this line in version 15 of the diff

changed this line in [version 15 of the diff](/AlekSIS/official/AlekSIS-App-Lesrooster/-/merge_requests/110/diffs?diff_id=68396&start_sha=520895f3847754716b3068208565e0ffe7cdafe3#c43f57521338b8464b72e54e5b89ae888d1510d4_508_502)
Author
Member

changed this line in version 15 of the diff

changed this line in [version 15 of the diff](/AlekSIS/official/AlekSIS-App-Lesrooster/-/merge_requests/110/diffs?diff_id=68396&start_sha=520895f3847754716b3068208565e0ffe7cdafe3#c43f57521338b8464b72e54e5b89ae888d1510d4_923_915)
Author
Member

changed this line in version 15 of the diff

changed this line in [version 15 of the diff](/AlekSIS/official/AlekSIS-App-Lesrooster/-/merge_requests/110/diffs?diff_id=68396&start_sha=520895f3847754716b3068208565e0ffe7cdafe3#c43f57521338b8464b72e54e5b89ae888d1510d4_960_951)
Author
Member

changed this line in version 15 of the diff

changed this line in [version 15 of the diff](/AlekSIS/official/AlekSIS-App-Lesrooster/-/merge_requests/110/diffs?diff_id=68396&start_sha=520895f3847754716b3068208565e0ffe7cdafe3#f31e49512b22ce63cd8cde616e9f49ac1fdba45d_222_222)
Author
Member

marked the checklist item Use permissions for all GraphQL mutations as completed

marked the checklist item **Use permissions for all GraphQL mutations** as completed
Author
Member

added 4 commits

Compare with previous version

added 4 commits <ul><li>35587779 - Remove quota variable</li><li>4dbd3542 - Remove item moved to teachers method</li><li>b36aba99 - Remove solved todos</li><li>bfb7aa6f - Use correct perms</li></ul> [Compare with previous version](/AlekSIS/official/AlekSIS-App-Lesrooster/-/merge_requests/110/diffs?diff_id=68396&start_sha=520895f3847754716b3068208565e0ffe7cdafe3)
Author
Member

added 1 commit

  • 8a16850b - Make supervision planning usable for users with global permission

Compare with previous version

added 1 commit <ul><li>8a16850b - Make supervision planning usable for users with global permission</li></ul> [Compare with previous version](/AlekSIS/official/AlekSIS-App-Lesrooster/-/merge_requests/110/diffs?diff_id=68404&start_sha=bfb7aa6f1b21817722dcd8518165e188d7fa83ad)
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

added 1 commit

Compare with previous version

added 1 commit <ul><li>bc7fbe29 - Reformat</li></ul> [Compare with previous version](/AlekSIS/official/AlekSIS-App-Lesrooster/-/merge_requests/110/diffs?diff_id=68406&start_sha=8a16850b7f439aa6ee2dff41435b8dd238cb07d8)
Owner

mentioned in commit 9c065da981

mentioned in commit 9c065da9813b27cea847e119c68acac7687d1bb7
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-Lesrooster!198
No description provided.