Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

frontend: fix teacher delete bug in admin panel #355

Closed
bsilkyn opened this issue Apr 19, 2024 · 0 comments · Fixed by #364
Closed

frontend: fix teacher delete bug in admin panel #355

bsilkyn opened this issue Apr 19, 2024 · 0 comments · Fixed by #364
Assignees
Labels
bug Something isn't working frontend Issues regarding the frontend prio: medium

Comments

@bsilkyn
Copy link
Contributor

bsilkyn commented Apr 19, 2024

There is a bug in admin panel, where if you delete the teacher role, the student role gets deleted instead.

@bsilkyn bsilkyn added this to the Final milestone milestone Apr 19, 2024
@bsilkyn bsilkyn self-assigned this Apr 19, 2024
bsilkyn added a commit that referenced this issue Apr 19, 2024
@bsilkyn bsilkyn added bug Something isn't working frontend Issues regarding the frontend labels Apr 20, 2024
bsilkyn added a commit that referenced this issue Apr 20, 2024
bsilkyn added a commit that referenced this issue Apr 21, 2024
BramMeir pushed a commit that referenced this issue Apr 24, 2024
* fix: fixed bug where deleting teacher role would delete the student role #355

* refactor: re-center search icons + use multiselect for role filtering + update global select #355

* chore: get data table out of Title component and put into Body component

* fix: proper translation of roles in admin table

* refactor: lint fixes

* fix: accidentally deleted I18n as import, now fixed
DeLany123 pushed a commit that referenced this issue Apr 29, 2024
* fix: fixed bug where deleting teacher role would delete the student role #355

* refactor: re-center search icons + use multiselect for role filtering + update global select #355

* chore: get data table out of Title component and put into Body component

* fix: proper translation of roles in admin table

* refactor: lint fixes

* fix: accidentally deleted I18n as import, now fixed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working frontend Issues regarding the frontend prio: medium
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant