Resolve "Add absence approval feature" #176
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-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
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
aleksis/AlekSIS-App-Kolego!176
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "55-add-absence-approval-feature"
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?
Information and screenshots
No further information have been provided.
Checklist
tox -e reformat,lintusedapps.pyandREADME.rstup-to-dateBackend checklist
Frontend checklist (mandatory with ~"part::frontend")
Closes #55
assigned to @yuha
added 1 commit
ed3cf29d- Add absence reason fields for approvalCompare with previous version
added 1 commit
2867fd86- Add absence reason approval fields to frontendCompare with previous version
added 1 commit
b6df6d7b- Add uuid field for groupCompare with previous version
added 1 commit
bdf1f8f4- Make absence reason layout clearer & add hintCompare with previous version
added 1 commit
1eb17940- Add todoCompare with previous version
added 1 commit
86d380cb- Use attendance class for absence personsCompare with previous version
added 1 commit
Compare with previous version
added 1 commit
109e3097- Implement attendee by status methodCompare with previous version
added 1 commit
2f53b2f9- Further adapt methods of Absence to attendanceCompare with previous version
added 1 commit
c5d55891- Add migration for attendance fieldCompare with previous version
added 1 commit
3e367bc9- Query attendance instead of person for absencesCompare with previous version
added 1 commit
4175dd05- Add todoCompare with previous version
added 1 commit
5a9b0245- Adapt filtersCompare with previous version
added 1 commit
c48469ca- Directly use imported Attendance in FK definitionCompare with previous version
added 1 commit
2e2c5975- Explicitly declare person attendance fieldCompare with previous version
added 1 commit
3a0c859d- WIP: implement attendance handling in absence mutationsCompare with previous version
added 1 commit
6a47208a- Fix usage of person in save functionCompare with previous version
added 1 commit
Compare with previous version
added 1 commit
f5b08b4e- WIP: use correct type for attendance haandling in mutationCompare with previous version
added 1 commit
3b2a1963- Refine absence RSVP logicCompare with previous version
added 1 commit
135ae7b9- add todoCompare with previous version
added 1 commit
0906a79d- Adapt predicatesCompare with previous version
added 2 commits
b51ba7f2- Add is_own_absence predicatec2534b6e- Use new predicate for absence creation ruleCompare with previous version
added 1 commit
b63f27f0- Fix usage of timezones in overlap handlingCompare with previous version
added 1 commit
aaf74cea- Add some todo/fixme commentsCompare with previous version
added 1 commit
ce5a5a0b- Really fix automatic FK attendance handlingCompare with previous version
added 2 commits
aa68b9ff- Query more person fields for attendance0b50b91d- Add display component for absence reasonCompare with previous version
added 1 commit
d7558c88- Drop inline default components in absences tableCompare with previous version
added 1 commit
98239226- Drop unused importCompare with previous version
added 1 commit
1bef460f- Add basic absence approval pageCompare with previous version
added 1 commit
c63996c6- Add approval buttons in actions column of CRUD listCompare with previous version
added 1 commit
Compare with previous version
added 1 commit
18db3088- Add todosCompare with previous version
added 2 commits
6ca66d04- Add predicate to check for membership in approval groups56ec568d- Add rule for approving absencesCompare with previous version
added 1 commit
c8388726- Add method ob absence class to check ruleCompare with previous version
added 1 commit
88e71cb4- Fix core migration referenceCompare with previous version
added 1 commit
ae8a5ea7- Use OneToOneField for attendanceCompare with previous version
added 1 commit
63069636- Integrate mutations into absence approval frontend and implement cache updatingCompare with previous version
added 1 commit
cc0ff8e0- Use dedicated query for absence approval pageCompare with previous version
added 1 commit
a758724a- Drop deprecated todosCompare with previous version
added 1 commit
7349c871- Add filtersCompare with previous version
added 1 commit
76bb8e10- Add todoCompare with previous version
added 1 commit
2a68f230- Disable patch for absence approval pageCompare with previous version
added 1 commit
7d873053- Set NEEDS-ACTION as default attendance status for absence requiring approvalCompare with previous version
For creating, at least check the person being used & don't only use global perms
-> maybe: make configurable who can create absences with a certain reason (or just allow creation for all approval-based absence types, at least?)
For editing:
main objective: once absences are created, there has to be a differentiation between edits:
if the absence is a "no-approval-needed" one, just do as before (or even allow edits of own absences; maybe: introduce some more elaborated logic of filtering which AbsenceTypes are allowed to be used by whom already at point of creation?)
if the absence is an "approval-needed" one, each edit at least has to reset the approval status OR be denied if performed by a non-administrator person who is allowed to create certain absences (but some marker of privileged positions has to be retained, like now, with the GroupType based approach)
maybe: some fields that can be edited by all persons involved without having to deal with the approval status? like the comment?
added 1 commit
83fdb5fa- Drop some todosCompare with previous version
added 1 commit
fb7f0d9b- WIP: Add methods for f/b statusCompare with previous version
added 1 commit
0271c7b0- Add managersCompare with previous version
added 1 commit
Compare with previous version
added 2 commits
2044877a- Refactor params handling in get_objects and only show accepted absences per default33d82b3c- Add freebusy type propertyCompare with previous version
added 1 commit
51c5d723- Complete get_availability_objectsCompare with previous version
added 1 commit
8a38b003- Show absence feed in calendar overview and expose via CalDAVCompare with previous version
added 1 commit
75c0ac18- Add missing importCompare with previous version
added 1 commit
99bfbf5c- Fix get_objects argumentsCompare with previous version
added 1 commit
60e9246d- Add missing importCompare with previous version
added 1 commit
ab0f1cae- Fix params handlingCompare with previous version
added 2 commits
22eb6f91- Fix resolve_planned_absences_for_person filter8be332c7- Adapt planned absences queryCompare with previous version
added 1 commit
2e7e202b- Fix absence widget handling of full day absencesCompare with previous version
added 1 commit
6d5aeab1- Show absence status in widgetCompare with previous version
added 1 commit
7b0aecfc- Allow non-admin users to access & create absencesCompare with previous version
added 1 commit
b36d2781- Drop todoCompare with previous version
added 1 commit
3116f7cb- Use better wording in status chipCompare with previous version
added 1 commit
2fab0d66- Fix overlap handling with duplicated attendancesCompare with previous version
added 1 commit
bec66f80- Adapt testsCompare with previous version
added 1 commit
26817f4c- ReformatCompare with previous version
added 1 commit
f359b278- Aadapt migrationCompare with previous version
added 1 commit
de68cd0a- Fix migrationsCompare with previous version
added 1 commit
151a100c- Add changelogCompare with previous version
added 1 commit
e832337c- Fix menu rulesCompare with previous version
marked this merge request as ready
added 1 commit
1d875539- ReformatCompare with previous version
assigned to @hansegucker and unassigned @yuha
added 1 commit
f170b2a7- Fix tox migration creationnCompare with previous version
mentioned in commit
046f0d351d