New data model based on calendar events #605
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!605
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "new-model-based-on-calendar"
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?
added 2 commits
6995c47b- Add more detailed properties to LessonEventa71d04c2- Add SupervisionEvent model for supervisionsCompare with previous version
mentioned in issue AlekSIS#58
added 2 commits
1e979f87- Reformat6a6159a6- Support substitutions with CalendarEvent (status and amends)Compare with previous version
added 1 commit
21bd3d8d- Provide current request to all value_ and get_ methods of calendar eventsCompare with previous version
added 1 commit
05dcffa9- Make descriptions of LessonEvents compatible with substitutionsCompare with previous version
FIXME Parent groups instead of groups in certain situations
FIXME Supervisions shown two times and both times not correct
added 2 commits
084d3e00- Fix group names in lesson event description4d6a3ded- Customize event bar and event details for lesson eventsCompare with previous version
added 1 commit
609ea368- Create migration for lesson eventsCompare with previous version
added 2 commits
7010fa49- Fix displaying of time in calendar events69035127- Merge branch 'new-model-based-on-calendar' of...Compare with previous version
added 1 commit
6fea5788- Ensure that all content of LessonEventBar is in the event barCompare with previous version
added 1 commit
b5b6feaa- Fix migration historyCompare with previous version
added 16 commits
master0b64ddc3- Merge branch 'master' into new-model-based-on-calendarb1b46778- Update common used mixins for groups, teachers, and roomsCompare with previous version
added 1 commit
59b489d8- Add missing migrationCompare with previous version
added 1 commit
a1d84b63- Add view for all timetables with lesson eventsCompare with previous version
added 1 commit
9dff100a- Show timetables in new calendar frontendCompare with previous version
added 1 commit
ad65d9c6- Move LessonEvent queries to own queryset and drop dangerous attributesCompare with previous version
added 1 commit
d8671616- Fix LessonEventQuerySetCompare with previous version
added 3 commits
d847e079- Adapt to Core 4.0 managed_by featured8b0f606- Merge branch '210-adapt-to-core-4-0' into new-model-based-on-calendareebb1823- Fix migration historyCompare with previous version
added 1 commit
ce4a8838- Fix teacher timetableCompare with previous version
added 2 commits
c9772b45- Drop import of CalendarWithControls3f75b026- Merge branch 'new-model-based-on-calendar' of...Compare with previous version
Cursus missing
added 1 commit
9e492da7- Add get_objects method for SupervisionsCompare with previous version
Any idea on this? ENGS has this problem.
added 3 commits
d48546a7- Add mobile view for timetables and improve select for timetables459937ca- Reformat8a0dc2fc- Merge branch 'new-model-based-on-calendar' of...Compare with previous version
added 1 commit
9bcdca79- Fix col widths in new timetable frontendCompare with previous version
added 1 commit
c07bd892- Add missing translationsCompare with previous version
Bug: Navigating to timetable works in frontend. But directly navigating to specific timetable url leads to an error in log and generic timetable frontend is shown. Tested & reproducible with url: http://localhost:8000/app/chronos/timetable/teacher/1/
JS console shows error:
Uncaught (in promise) NavigationDuplicated: Avoided redundant navigation to current location: "/app/chronos/timetable/teacher/1/".
createRouterError vue-router.esm.js:2046
createNavigationDuplicatedError vue-router.esm.js:2016
confirmTransition vue-router.esm.js:2333
transitionTo vue-router.esm.js:2260
push2 vue-router.esm.js:2606
push vue-router.esm.js:3036
push vue-router.esm.js:3035
selected Timetable.vue:34
VueJS 12
click SelectTimetable.vue:44
VueJS 4
click VListItem.ts:126
VueJS 33
vue-router.esm.js:2046:14
Based on group type?
.
FIXME: My timetable does not redirect to my timetable
just remove this menu entry (and all the other legacy ones)
@permcu could you do that?
@permcu could you do that?
added 2 commits
a6946c66- Remove legacy menu entries8b7a9d87- Add permission checkCompare with previous version
added 1 commit
c8e10b49- Show selected timetable according to route parametersCompare with previous version
FIXME: Differentiate permissions for timetables into group/teacher/room permissions (like it was before)
FIXME: Chronos parent menu point is shown regardless of permissions (even though a user is not allowed to access any timetables).
FIXME: Calendar auto refresh fetches wrong data when the user has switched timetables beforehand, leading to the lessons/supervisions of the first accessed plan being displayed after the first refetch. @yuha
FIXME: Complete German translations
FIXME: EventBars/Details for supervisions are not differentiated enough from lessons
FIXME: Contrast in some cases between text and subject colour is suboptimal (yellow lesson; white text)
Fixed with: https://edugit.org/AlekSIS/official/AlekSIS-Core/-/merge_requests/1324
to my knowledge those are finished?
added 1 commit
10b5781b- Use existing permissionsCompare with previous version
Yeah, well that depends on setting the subject's colors?
added 10 commits
masteref150b95- Merge branch 'master' into new-model-based-on-calendar1d3d4ead- Merge branch 'new-model-based-on-calendar' of...0947c567- Update pyproject.toml25585b51- Fix lintCompare with previous version
added 1 commit
fa1706c9- Drop outdated regression testCompare with previous version
added 1 commit
c2a4ccb8- Bump versionCompare with previous version
created #218 to continue this discussion
mentioned in issue #218
created #219 to continue this discussion
mentioned in issue #219
created #220 to continue this discussion
mentioned in issue #220
added 2 commits
3e10726d- Correctly show supervisions in the calendar315680b9- Add supervisions to timetableCompare with previous version
Use generalized core components (see https://edugit.org/AlekSIS/official/AlekSIS-Core/-/merge_requests/1330) for timetable overview
changed this line in version 31 of the diff
added 33 commits
masterefeed7b3- Merge branch 'master' into new-model-based-on-calendarCompare with previous version
added 1 commit
e105841a- Fix migrationsCompare with previous version
added 1 commit
Compare with previous version
added 1 commit
89e7d348- Allow filtering LessonEvents by courseCompare with previous version
added 1 commit
Compare with previous version
added 1 commit
6552e7fc- Add graphene type for LessonEventCompare with previous version
Needs proper review & decision about how to deal with the legacy frontend
requested review from @nik
assigned to @hansegucker
@yuha do this!
Hard dependency on Cursus?
Why?
Only teachers?
No newline at end of file
No newline at end of file
.
Yes, that's correct. That's why Cursus is a separate app so we don't depend on Lesrooster.
Because this refetches the amended LessonEvent each time (the actual amends attribute always returns a normal
CalendarEventobject).There were some privacy concerns with sharing the students list over the calendar API.
changed this line in version 37 of the diff
changed this line in version 37 of the diff
added 1 commit
4a7d8da4- Fix "No newline at end of file"Compare with previous version
added 1 commit
750ff75a- Allow filtering for amended and amending LessonEvents in get_objectsCompare with previous version
added 1 commit
d1af59cf- Allow filtering for own lessons as teacherCompare with previous version
added 1 commit
fdd0d935- Bump dev versionCompare with previous version
added 1 commit
Compare with previous version
added 1 commit
Compare with previous version
added 1 commit
Compare with previous version
added 1 commit
Compare with previous version
added 1 commit
ae6f14ac- Fix get_objects filtering for all/own lessonsCompare with previous version
added 2 commits
e1695a38- Translate timetable types in frontende028f57d- ReformatCompare with previous version
changed this line in version 47 of the diff
added 1 commit
63826a5d- Show all groups in timetable viewCompare with previous version
Then why don't put the cursus stuff directly in chronos? is there a use case where one would use cursus without chronos?
mentioned in issue #226
added 1 commit
955800d1- Specify active icon for chronosCompare with previous version
Yes.
This
amendedis never used again. @yuhaadded 2 commits
29689fd6- Fix select-timetable value prop typedcc6a604- Merge branch 'fix-select-timetable-prop' into 'new-model-based-on-calendar'Compare with previous version
@yuha wants to do that, but in a separate issue: https://edugit.org/AlekSIS/official/AlekSIS-App-Chronos/-/issues/230
Removal of frontend will happen in https://edugit.org/AlekSIS/official/AlekSIS-App-Chronos/-/issues/231
https://github.com/jazzband/django-polymorphic/issues/437
changed this line in version 50 of the diff
added 1 commit
037639b9- Remove real_amends as its longer necessary (amends is fixed!)Compare with previous version
Please investigate. I would like to close this MR.
added 1 commit
92e3861c- Add docstrings and typing for lesson event, fix request handlingCompare with previous version
added 2 commits
00ac1913- Fix managers07bd0b50- ReformatCompare with previous version
resolved all threads
@hansegucker Everything should be solved by now. Please have a look again.
marked this merge request as ready
added 1 commit
47cc59ba- Update changelogCompare with previous version
mentioned in commit
ccbc6ee085