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

Add a check to 'Create workshop' logic to see if categories are correct #151

Closed
DmyMi opened this issue Jun 9, 2021 · 0 comments · Fixed by #157
Closed

Add a check to 'Create workshop' logic to see if categories are correct #151

DmyMi opened this issue Jun 9, 2021 · 0 comments · Fixed by #157
Assignees
Labels
backend Issue involves backend functionality task General technical task

Comments

@DmyMi
Copy link
Contributor

DmyMi commented Jun 9, 2021

During 'Create workshop' logic check to see if categories are correct, i.e. there is a valid category with this id, a valid subcategory id within that category and a valid subsubcategory id within subcategory.

@DmyMi DmyMi added the task General technical task label Jun 9, 2021
@OlhaHoliak OlhaHoliak self-assigned this Jun 10, 2021
OlhaHoliak added a commit that referenced this issue Jun 24, 2021
* Sorted: mapping methods, startup services

* Added validation of provider rights

* Create and update workshop with inner entities

* WorkshopController tests

* Add transactions to WorkshopService

* Workshop service Tests

* Refactored StatusCodes, add controller test

* added application deletion

* Documentation. Added a file path to comments XML-file for Swagger

* Fix some code according to csharp_style
@OlhaHoliak OlhaHoliak linked a pull request Jul 1, 2021 that will close this issue
@OlhaHoliak OlhaHoliak added the backend Issue involves backend functionality label Jul 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend Issue involves backend functionality task General technical task
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants