Resolve "Take teachers' availability status into account in substitution planning" #741
No reviewers
Labels
No labels
Security
TeX
auto-update
board
done
board
ready
board
todo
check
delete-eslint-rc-js
check
update-builddeps-package-json
check
update-eslint-rc-js
check
update-gitignore
check
update-merge-request-template
check
update-prettier-ignore
check
update-pyproject-toml
check
update-renovate-json
check
update-tox-ini
part
backend
part
ci
part
docs
part
frontend
part
i18n
part
non-technical
part
packaging
prio
1
prio
2
prio
3
release-mr-5.x
size
large
size
medium
size
small
source
customer
source
customer::fsmw
source
customer::fss
source
customer::teckids
source
downstream
type
breaking
type
bug
type
feature
type
refactoring
workflow
blocked
workflow
confirmed
workflow
current-todo
workflow
discussing
workflow
new-app
workflow
wontfix
source::aleksolotl
No milestone
No project
No assignees
2 participants
Notifications
Due date
No due date set.
Dependencies
No dependencies set
Reference
aleksis/AlekSIS-App-Chronos!741
Loading…
Reference in a new issue
No description provided.
Delete branch "290-take-teachers-availability-status-into-account-in-substitution-planning"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Closes #290
assigned to @yuha
added 6 commits
7d225bc6- Include LessonEvents in FreeBusyEvent feedee5bd730- Handle absences correctly (?)40f58929- Add str method to LessonEvent906b26b6- Finally fix f/b aggregation handling of amended lessonsc3aaa658- Reformatbd0ad225- (Probably) make substitution method fasterCompare with previous version
added 1 commit
8513af0a- Pass availability time range to teacher fieldCompare with previous version
added 2 commits
a407eb72- Exclude cancelled lesson events from f/b feed76cc9a4b- Merge branch '247-use-lessonevent-feed-for-aggregated-availability-feed' into...Compare with previous version
added 1 commit
71c58cfb- Fix substitution planning page for non-admin usersCompare with previous version
added 1 commit
b7d9ebb4- ReformatCompare with previous version
marked this merge request as ready
@hansegucker please give this a test; somehow it seems to me that after dropping the detour via the legacy calendar feed generation mechanism and instead resorting directly on get_occurrences this became slower?
assigned to @hansegucker and unassigned @yuha
added 1 commit
a5cba618- Implement priority availability typeCompare with previous version
added 1 commit
428de686- ReformatCompare with previous version
Same as in the other MR
That distinct shouldn't be necessary.
forget about that, at least its not slow anymore for me. don't know how this occured
mentioned in merge request AlekSIS-Core!2173
changed this line in version 8 of the diff
added 2 commits
7107de33- Use more efficient mechanism for filtering lesson events relevant for free/busy status68b31ebe- Merge branch '247-use-lessonevent-feed-for-aggregated-availability-feed' into...Compare with previous version
changed this line in version 9 of the diff
added 1 commit
46e43a27- Remove unused distinct callCompare with previous version
resolved all threads
added 39 commits
master103b6f62- Merge branch 'master' into...Compare with previous version
mentioned in commit
c25ff8a8c9