Frontend #20
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-Maka!20
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "frontend"
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 #4
Closes #2
assigned to @ZugBahnHof
added 21 commits
466389d6- Fix casing typo in effort mutation fields678c2855- Mark required fields as required5d3b1659- Add field grades to efforts gql type0d4cb746- Generic chip to display grades7a3ddb37- Fix gql queries for gradechoices71b3be06- Display efforts on group page0b72230e- Simplify table14c0b9e8- More translations983312fd- UpdateOrCreateGrade.vue grade selectorc727ca51- Use UpdateOrCreateGradeCompare with previous version
added 8 commits
1d13e9f6- Create gradesa4a9ad5f- Create Create effort component78a00097- Allow creation of efforts255f966b- Fix infinite render loop bugd58d27c4- Use autocomplete instead of select92157bd3- Disable groups for effort creation and update cache correctlyb8efb3b8- Reduce visual load of grade bookb8c07823- Allow tabbing column-wise through the tableCompare with previous version
deleted the
1-new-app-gradebookbranch. This merge request now targets themainbranchchanged the description
this now includes the shared secret mechanism
added 17 commits
752416ea- 1 commit from branchmainc67e68ac- Merge branch 'frontend' into 2-implement-shared-secret-mechanism8f81d7f0- Refactor shared secret related mixinsd6211984- Fix checking mechanism for session attribute86503e41- Move shared secret component8aefd1a9- Rename shared secret wrapperf92f2cfd- Add shared secret wrapper to all pagesde1a1135- Add translation for checking shared secretca035885- Re-add shared secret submitting mutation3b481310- Do not load shared secred wrapper slot content before initial query check is finished1fe1ef97- Use correct toolbar keysCompare with previous version
FIXME Real permissions
as you return
booltypeThis mutation probably should not return ok if the secret is wrong
Grade sets don't need the shared secret.
Effort types don't need a shared secret.
Use menu permission
FIXME permission
FIXME permission
FIXME permission
\ No newline at end of file
Should not be necessary to refetch, the status should come as response from the mutation and the cache of
sharedSecretStatusshould be updated accordingly.\ No newline at end of file
Debug log
TODO
\ No newline at end of file
\ No newline at end of file
TODO
\ No newline at end of file
\ No newline at end of file
\ No newline at end of file
\ No newline at end of file
TODO
\ No newline at end of file
\ No newline at end of file
\ No newline at end of file
\ No newline at end of file
\ No newline at end of file
changed this line in version 5 of the diff
changed this line in version 5 of the diff
added 1 commit
3d6464a8- Remove unneeded shared secret ruleCompare with previous version
changed this line in version 6 of the diff
changed this line in version 6 of the diff
added 1 commit
60442f8e- Directly check if shared secret is correct in mutationCompare with previous version
added 1 commit
76af73d6- Skip queries of efforts overview page until correct shared secret is enteredCompare with previous version
added 1 commit
b6e05f09- Use correct name for effort related mutationsCompare with previous version
changed this line in version 9 of the diff
changed this line in version 9 of the diff
added 1 commit
cf1f2c97- Use separate components for wrapping components that make queries on their ownCompare with previous version
added 5 commits
228740fb- Order grade choices per default36d6912c- Merge remote-tracking branch 'origin/frontend' into frontend83e79352- Merge remote-tracking branch 'origin/frontend' into frontendea964229- Redesign shared secret wrapper209c00cd- Merge remote-tracking branch 'origin/frontend' into frontendCompare with previous version
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
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
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
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
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 3 commits
1c392c90- Use current version of build config files34cb5b72- Reformat automatically652b24d1- Reformat manuallyCompare with previous version
changed this line in version 12 of the diff
added 1 commit
289404f4- Remove debug logCompare with previous version
changed this line in version 13 of the diff
added 6 commits
4cce9316- Allow updating of gradesb81fd77c- Dsiplay loading state37a6d111- Fix styling of SharedSecretWrapper.vue if secret is correct611075a0- Allow deletion of grades7d6cdcd3- Reformat24eecd84- Use change event instead of inputCompare with previous version
see https://edugit.org/AlekSIS/onboarding/AlekSIS-App-Maka/-/merge_requests/4
see https://edugit.org/AlekSIS/onboarding/AlekSIS-App-Maka/-/merge_requests/4
see https://edugit.org/AlekSIS/onboarding/AlekSIS-App-Maka/-/merge_requests/4
see https://edugit.org/AlekSIS/onboarding/AlekSIS-App-Maka/-/merge_requests/4
see https://edugit.org/AlekSIS/onboarding/AlekSIS-App-Maka/-/merge_requests/4
status now already comes from mutation; TODO: update cache
can't we just use the weight property?
Color picker does 8-digit hex, backend wants 6-digit
IDs?
Auto select first entry in autocomplete
Why is there a red border?
Ugly here
This is the color of the effort type you defined
Need to be removed
There is an issue for this in the Aleksis core
The question is not, what the correct value is (would be the
valueof the grade property btw), but how to access it...Wouldn't it be great to include the effort type (color) in the heading of the column?
changed this line in version 14 of the diff
changed this line in version 14 of the diff
changed this line in version 14 of the diff
changed this line in version 14 of the diff
added 15 commits
2c34edf3- Actually allow for filtering grades & efforts by group ID as needed by group overview page9eca127e- Include subgroups in effort/grade query filtering01b96b63- Remove shared secret check from grade set and effort type types/mutations/pagesf88817ba- Apply 2 suggestion(s) to 1 file(s)75681d41- Rename rules9ec6b33d- Fix get_queryset with shared secret mixin39d0aa7c- Fix permission filtering for viewing own grades980c5cee- Separate fetch and view rulesb9a63416- Reformata56a2fc6- Merge branch '9-add-permissions-rules' into 'frontend'Compare with previous version
added 1 commit
cd4af30d- Add fixture for common German grade systemsCompare with previous version
added 1 commit
848dd22b- Add translation filesCompare with previous version
added 2 commits
6db0d464- Translated using Weblate (German)922c028e- Merge branch 'weblate' into 'frontend'Compare with previous version
added 3 commits
613d2099- Added translation using Weblate (German)bb76873e- Translated using Weblate (German)48769ceb- Merge branch 'weblate-frontend' into 'frontend'Compare with previous version
Is this important right now @hansegucker ?
I think for that we should wait until we can assign icons to effort types
added 2 commits
2a71c364- Remove id columnecd5e77f- Force flat shared secret wrapperCompare with previous version
@yuha please have a look
@yuha whats the status on this?
created #10 to continue this discussion
mentioned in issue #10
added 4 commits
1c5f8e57- Show efforts of first-level child groups14013823- Auto-select first grade96b992c8- Translate grade removal confirmation dialogc4a61263- Display message for empty effortsCompare with previous version
added 1 commit
dbaa5541- Show message if group is a child groupCompare with previous version
@yuha can you do this please?
i need to check
no, the cache of the
sharedSecretStatusquery isn't getting updated yet. I wouldn't consider this to be particularly important, but still as nice to have. Could we just put that into a follow-up issue?TODO: check that permissions are checked properly in for
can_edit/can_deletegql type fields.
TODO: Add preference that switches whether parent group owners may edit grades/efforts of child groups
changed this line in version 22 of the diff
added 1 commit
763b8149- Add default value when getting shared secret status from sessionCompare with previous version
changed this line in version 23 of the diff
added 1 commit
2796c21f- Check for permissions when displaying create effort buttonCompare with previous version
Yes
I don't know, I didn't write TODO there.
Follow up issue please
added 1 commit
9af966ad- Make child group effort/grade viewing/managing configurableCompare with previous version
added 1 commit
Compare with previous version
mentioned in issue #12
https://edugit.org/AlekSIS/onboarding/AlekSIS-App-Maka/-/issues/12
added 1 commit
3a79b9f7- ReformatCompare with previous version
created #13 to continue this discussion
resolved all threads
mentioned in issue #13
requested review from @hansegucker
assigned to @hansegucker and unassigned @ZugBahnHof
marked this merge request as ready
mentioned in commit
867262a617