Resolve "Review permissions" #417
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
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
aleksis/AlekSIS-App-Chronos!417
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "83-review-permissions"
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 #83
added 1 commit
Compare with previous version
added 1 commit
Compare with previous version
added 1 commit
67685b61- Add additional rules&permissions in order to differentiate different access to...Compare with previous version
assigned to @hansegucker
assigned to @nik and unassigned @hansegucker
requested review from @hansegucker
It might be more efficient I we exclude objects without permission already in the view (by filtering the queryset like we did it in Alsijil).
It might be more efficient I we exclude objects without permission already in the view (by filtering the queryset like we did it in Alsijil).
It might be more efficient I we exclude objects without permission already in the view (by filtering the queryset like we did it in Alsijil).
@yuha At first glance, this already looks very fine. Can you please merge the current master into your branch and make the pipeline finishing successful? Please, check also whether this MR is still draft.
added 55 commits
master229f75f1- Merge branch 'master' into 83-review-permissionsCompare with previous version
added 1 commit
5270d4d2- Fix order of importsCompare with previous version
changed this line in version 6 of the diff
added 1 commit
87f5b9e3- Apply 1 suggestion(s) to 1 file(s)Compare with previous version
changed this line in version 7 of the diff
added 1 commit
324305e8- Apply 1 suggestion(s) to 1 file(s)Compare with previous version
changed this line in version 8 of the diff
changed this line in version 8 of the diff
changed this line in version 8 of the diff
added 3 commits
baece794- Move filtering of timetables from template to views725c9990- Reformatf0b07d20- Merge remote-tracking branch 'origin/83-review-permissions' into 83-review-permissionsCompare with previous version
resolved all threads
marked this merge request as ready
marked this merge request as draft
assigned to @yuha and unassigned @nik
added 35 commits
master8769661a- Merge branch 'master' into 83-review-permissionsCompare with previous version
marked this merge request as ready
added 2 commits
1618796e- Use custom queries to speed up permission filtering of timetables in overview04626927- Redo importsCompare with previous version
Please factor the three branches of the big predicate out into threee predicates
changed this line in version 11 of the diff
changed this line in version 11 of the diff
changed this line in version 11 of the diff
added 1 commit
Compare with previous version
resolved all threads
added 1 commit
3cfa597b- Reformat predicatesCompare with previous version
added 38 commits
master4d2ed6e3- Merge branch 'master' into 83-review-permissionsCompare with previous version
added 1 commit
Compare with previous version
added 1 commit
ebd013b7- Reformat docstringsCompare with previous version
assigned to @nik and unassigned @yuha
Please add a docstring
Please add a docstring
Please add a docstring
added 1 commit
c4ccb64d- Add docstringsCompare with previous version
resolved all threads
changed this line in version 17 of the diff
changed this line in version 17 of the diff
changed this line in version 17 of the diff
added 1 commit
Compare with previous version
added 2 commits
Compare with previous version
aborted the automatic merge because source branch was updated
added 2 commits
d0355a61- Use optional type annotationsCompare with previous version
aborted the automatic merge because source branch was updated
added 1 commit
5f803edc- Fix get_classes querysetCompare with previous version
mentioned in commit
186db4706a