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.

Sign up for GitHub

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

Jump to bottom

Deleting direction, department or class #530

Closed
DmyMi opened this issue Jan 26, 2022 · 0 comments · Fixed by #534
Closed

Deleting direction, department or class #530

DmyMi opened this issue Jan 26, 2022 · 0 comments · Fixed by #534
Assignees
@alrom @Sandaljero
Labels
backend Issue involves backend functionality enhancement New feature or request

Comments

@DmyMi
Copy link
Contributor

DmyMi commented Jan 26, 2022

Admin should not be able to delete direction, department or class if they have at least 1 workshop attached.
If there are no workshops attached - admin should be able to delete a direction and all related departments and classes should be deleted, as well as when deleting a department - it should delete all related classes.
@Sandaljero Sandaljero added backend Issue involves backend functionality enhancement New feature or request labels Jan 28, 2022
Sandaljero added a commit that referenced this issue Jan 28, 2022
@Sandaljero
#530 Admin should not be able to delete direction, department or clas…
c22a862 
…s if they have at least 1 workshop attached. Some tests were added.
@Sandaljero Sandaljero linked a pull request Jan 28, 2022 that will close this issue
#530 Romanchuk lukashov/deletion direction department class #534
Merged
Sandaljero added a commit that referenced this issue Feb 1, 2022
@Sandaljero
#530 Romanchuk lukashov/deletion direction department class (#534)
7963d7d 
* #530 Admin should not be able to delete direction, department or class if they have at least 1 workshop attached. Some tests were added.

* Small fix

* ; was deleted
@DmyMi DmyMi closed this as completed Feb 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@alrom alrom

@Sandaljero Sandaljero

Labels
backend Issue involves backend functionality enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

#530 Romanchuk lukashov/deletion direction department class ita-social-projects/OoS-Backend
3 participants
@alrom @DmyMi @Sandaljero