Resolve "Add export functionality to course book" #812
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-Alsijil!812
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "258-add-export-functionality-to-course-book"
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 #258
assigned to @hansegucker
added 72 commits
e68baa81- Respet `onlyShowCount` in ExtraMarkChip.vue9600f28f- Display chips for absence reasons and extramarks12b68860- Show tardinesses in group statistics table0b99184e- Simplify translations4c265ccf- Show (currently inactive) "Details" buttone77cfa11- Make chips dense and center aligned667648fc- Reformat765fc1bc- Improve statistic queries, fix some bugs, include tardinessf1aab9f2- Use person chip for person columnCompare with previous version
added 5 commits
733b485e- Merge branch master into 266-add-statistics-page-for-absences9af9c13c- Revert "Comment out task"Compare with previous version
added 2 commits
Compare with previous version
added 1 commit
Compare with previous version
added 3 commits
26ea069c- Split register-template and make it configurableb6a41a4d- Rename now configurable full_register to register_for_groupb0d967dc- Capture TODOsCompare with previous version
added 1 commit
Compare with previous version
added 5 commits
fbbc2e79- Wire register_for_group to url and thread parameters throughe47314b8- Adapt the register_teachers_and_subjects_table to the new datamodel68909450- Adapt the register_cover to the new datamodelCompare with previous version
added 5 commits
9c422c5b- Adapt the register_abbreviations to the new datamodel462b81f7- Properly use django template filters in teachers_and_subjects_tableb70dea9d- Adapt the register_members_table to the new datamodelb1e27b20- Reduce queries for teachers_and_subjects_table with prefetchCompare with previous version
added 1 commit
Compare with previous version
added 1 commit
91b88c4f- Adapt the person_overview to the new datamodelCompare with previous version
added 1 commit
12aafbf1- Fix and simplify person_overview filteringCompare with previous version
added 2 commits
aee8c0bd- DeduplicateCompare with previous version
added 1 commit
Compare with previous version
added 1 commit
Compare with previous version
added 11 commits
10f6a645- 1 earlier commit53245fb5- Adapt the firt-part of the coursebook-template to the new datamodelcccfa48a- Factor out the notable_participations and personal_notes prefetch2dc28493- Consider different related names for personal_notes41f66b0d- Expose select_related from participation and personal_note prefetch2fc91bab- Properly prefetch and sort documentations34e22655- Adapt the coursebook-template's absences to the new datamodela7aa9e28- Adapt the coursebook-template's peronal-notes to the new datamodel5061222c- Finish the coursebook-template4fa766a9- Tidy the coursebook-template351b377f- Cleanup old code from generate_full_register_printout in tasks.pyCompare with previous version
@permcu Please update with current changes from @ZugBahnHof
TODO
There is a method of Documentation, I think, that should do the formatting for you.
Same as above, but all teachers and name as fallback
Yeah, well, we ignore this for now.
Also would do this in one table and add
not counted as absentin the respective rowsTODO
This should use the color of the absence reason.
No, archive has to be complete.
All teachers!
Use blocktrans with placeholder
Yes
That doesn't seem to correct because the table will be empty if there aren't any groups.
Also use blocktrans with placeholder
Done. But his changes were unrelated to mine.
added 27 commits
c46a2427- Update statistical queries to reflect schema changesb88f7fd3- Use new queriesabfc8bc5- Query more documentation fields5b2d5b99- Allow mode swithcing on person statistics376c272a- Show notefbc06654- Show Rearrange list item title03cc18b2- Apply ripple effect to list itemsada3a164- Fix date rendering270ccd17- Print only absences with class_register as absence_reason_tags36061dd0- Merge branch origin/266-add-statistics-page-for-absencesCompare with previous version
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
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
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
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
changed this line in version 17 of the diff
added 1 commit
c601a021- Edit coursebook text-snippets - mostly casingCompare with previous version
changed this line in version 18 of the diff
added 1 commit
Compare with previous version
changed this line in version 19 of the diff
added 1 commit
Compare with previous version
changed this line in version 20 of the diff
added 1 commit
Compare with previous version
There is an extra group.name column if there are more than one group. This is intended to show child groups. If still unclear have a look a the prev. template. This table generalizes what prev. where two tables.
changed this line in version 21 of the diff
added 1 commit
Compare with previous version
added 1 commit
Compare with previous version
changed this line in version 23 of the diff
added 1 commit
Compare with previous version
How should count as absent be indicated then? Or no distinction?
It was missing already. I can choose something when I've seen the printout.
changed this line in version 24 of the diff
changed this line in version 24 of the diff
changed this line in version 24 of the diff
added 2 commits
Compare with previous version
I could not find such a method and created it myself now.
changed this line in version 25 of the diff
added 2 commits
Compare with previous version
Done & also did it everywhere else absence-reasons are printed.
added 2 commits
Compare with previous version
added 19 commits
1af576c1- Show absence_reasons in their colour in rest of coursebook-printout1c68c321- Rename coursebook-filters-component to coursebook-controls64724594- Add print button to coursebook4ed99604- Introduce a coursebook-print-dialog48b8e9c0- Add a title to the coursebook-print-dialog2016b446- Handle loading in the coursebook-print-dialogccb26fb3- Add group-selection to the coursebook-print-dialog5ecc6779- Add checkboxes for included parts to the coursebook-print-dialog9da714c5- Add buttons to the coursebook-print-dialogdfcef6af- Implement print-method for the coursebook-print-dialogCompare with previous version
added 9 commits
7d20b0e6- Fix route to coursebook-printoutc3f5cf28- Update import8d89bf54- Fix annotate_person_statistics_from_documentationsCompare with previous version
added 6 commits
d1465a0c- Change generate_full_register_printout to list of groupse288bad8- Change register_printout-template to list of groups1c3ea3ec- Properly provide group wrapped in list in template7d1220be- Remove TODOCompare with previous version
added 19 commits
65dcb1d3- Add include-switches to context2aeb5b53- Add load-tags to register-printout-subtemplates2b09fef0- Fix closing parenef34f1e9- Remove the python-tuple-trap8e13ec80- Make coursebook-print-dialog a bit prettier28f5098d- Add today to context427df430- Add missing import0cff6a3b- Properly parse get-params in full_register_for_group-view080a56f6- Default tardiness_sum to 0 if Nonea06da45a- Fix register_printout-templatesCompare with previous version
changed this line in version 31 of the diff
added 1 commit
5d92d9f6- Add missing loop over documentations in coursebook-printoutCompare with previous version
added 54 commits
masterb3e94ed8- Order statistics desc by time592425bc- Reformat13fbb415- Add (disabled) print button726d95d3- Apply 1 suggestion(s) to 1 file(s)048c091a- Merge branch 'master' into 266-add-statistics-page-for-absencesbe3ee4d5- Limit page widthe4fdd4c4- Move note to the sided4ea461e- Move statistics card to bottomsheet on mobile2e511a6c- Reformat39ebfb19- Merge 'origin/266-add-statistics-page-for-absences'Compare with previous version
No distinction
Debug print
changed this line in version 34 of the diff
changed this line in version 34 of the diff
changed this line in version 34 of the diff
added 1 commit
f3a4cf12- Format translationsCompare with previous version
added 15 commits
c07a966f- Fix spellinge82d8b2d- Add clarifying TODO6386ff6f- Fix typo3e796c4d- Fix prefetch_personal_notes45714ec8- Add another default-value for tardiness8337b07c- Guard against extra_mark is Nonebd28b2cc- Show absence-reasons in one table0456af8e- Edit extra-marks-heading5d773942- Put parens around extra-mark-short-names as wellf0923e32- Pass absence-reasons to person-overviewsCompare with previous version
changed this line in version 36 of the diff
added 1 commit
283cf2e0- Remove debug printCompare with previous version
added 1 commit
5f1e1fba- Remove reference to non-existing lessons-day-head-css-classCompare with previous version
resolved all threads
Here is a current register-printout:
register-printout.pdf
Unresolved issues:
If only a note the row is missing and if note and extra_mark only the extra_mark shows.
If there is an attached extra_mark the note is not shown at all.
Last two seem like a backend (tasks.py) issue. But I can not reproduce it at the shell. There it works just fine.
The looks are preserved from before.
Please have a look at personal overviews if they are still as intended.
I think the day in the coursebook should have date - but where. Or maybe add week indicators back in.
Also the AlekSIS-Logo is missing. I do not know why it does not show. The server serves it.
added 1 commit
96c01c8b- Tidy up coursebook exportCompare with previous version
added 33 commits
master34d510f9- Merge 'origin/master'Compare with previous version
added 2 commits
641ec995- Fix templates for coursebook exporta77056f6- Add validation to coursebook print dialogCompare with previous version
added 1 commit
24251792- Update changelogCompare with previous version
added 1 commit
Compare with previous version
added 1 commit
Compare with previous version
added 2 commits
925c6d5a- Fixup the tidyupa3e135c9- Fix indentationCompare with previous version
added 1 commit
f993c9b7- Fix the fixCompare with previous version
mentioned in merge request AlekSIS-Core!2053
Finally everything works:)
It now needs the MR in Core mentioned above.
register-printout.pdf
Still please have a look at the person-overview pages. Are they as intended? They seem misaligned for me & could still need some styling. (I just left them as I found them for now - apart from fixing table-page-break-issues)
@hansegucker Why the restrictions on which parts to print? The printout is fully modular = the user could print whatever he wants.
@hansegucker Please have a look at the changes to the coursebook date in the last commit. I prefer it. But not sure if should keep.
added 2 commits
971043c4- Fix the coursebook-print-dialog-validationa6e2a7f4- Add day-name to date display in coursebook-printoutCompare with previous version
Yeah, but he has to include at least one page with information (cover and abbreviations don't count).
Why? I would say: Power to the user. If she just needs a cover give her a cover.
resolved all threads
marked this merge request as ready
added 1 commit
fbca4c69- ReformatCompare with previous version
mentioned in commit
1b72b30971mentioned in issue #267