-
Notifications
You must be signed in to change notification settings - Fork 0
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
[RSN-14] - Create DB services #62
Merged
+4,159
−129
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…, role, user*) (#19) * Created services classes and added TODO methods for each class * Created basic CRUD methods * Changed methods returing types, checks for null * Checks if user already exists in db etc., created tests for services * Fixed unit test * Changed type taken by the services, new, improved tests for services * Final fixes and changed unit tests for methods with foreign keys * Changed delete method to return bool, fixed typos and other problems based on PR review * Added brackets one line ifs and added delete method for ObjectTypeService * Revert "Added brackets to one line ifs, updated delete method in ObjectTypeService" This reverts commit 1c69f11, reversing changes made to 1a58a27. * Created services classes and added TODO methods for each class * Created basic CRUD methods * Changed methods returing types, checks for null * Checks if user already exists in db etc., created tests for services * Fixed unit test * Changed type taken by the services, new, improved tests for services * Final fixes and changed unit tests for methods with foreign keys * Changed delete method to return bool, fixed typos and other problems based on PR review * Added brackets one line ifs and added delete method for ObjectTypeService * Updated services after scaffold Updated services and tests, User service not finished, enums not working * Workflow fixes * Unit test fix * Working user service * Deleted test controller * Transaction for user update service * Updated services and created mappers * Services throw errors Services now throw errors if something is not right (dto null, data not found in db), updated unit tests and created custom exception classes * Mappers function name change * Update based on review * Added GetUserByUsername function * Changed exceptions directory location
Loading status checks…
* Created CRUD services for Parameter, Status, Tag and C&U for Event * created CRUD Image, Interest, Event and Parameter Services Also added geting all elements and by filter(not done properly yet) * changed delete services to void isnted of bool * Fixed GetByFilter service in Event, Image, Intrest, Parameter, Status and Tag Services * little changes * fixing build issue and changing list to ienumerable * Fixing EventServices and standardizing syntax issues * created test for interest status parameter and tag * Created ImageServicesTests * Updated Parameter, Tag, Status services Checked if any of Parameters, Tags or Statuses are used before delating/updating. * fixed bug in imageservicetest * one more fix * Updated services and tests Changed delete methods to bool, updated mocks on tests. * Updated services * Updates after scafoled nr 1 * Updated services Changed using bridge tables to using colections in EF. * Updated EventService added a logic for parameters in event * working event services * created mappers and small fixes * finished mappers * Added exceptions in services * fixed mappers according to our convension * Changed exceptions directory location * fixed build error * Exceptions changes * Unit tests fix * updated services * Slug creating method updated * Update ImageService.cs * Update ImageService.cs * updated image services and tests also updated slug method for event services * event service delete method updated * refactor and fixes * refactor * Update EventService.cs * last refactor * removing spaces * added geteventbyslug method to eventservice * Update ImageService.cs * refactor * changed list to Enumerable * changed list to enumerable * folder refactor * updated services * Update EventService.cs * changes * Update EventServicesTest.cs * updates * Update EventService.cs * Update EventService.cs * Update ImageService.cs * Updates * Update EventMapper.cs * changes * Update ImageServiceTests.cs * changes * Formating * Update EventService.cs * Update EventService.cs * changes * Created AddEventComment method * Update EventService.cs * fixed updating and creating event * Update EventService.cs * changed create slug and text for event --------- Co-authored-by: Maciej Koperdowski <[email protected]>
mkoper02
added
scope: backend
Indicates that PR relates to backend changes.
type: feature
PR's categorised as feature.
labels
Jun 18, 2024
wzarek
added
the
status: ready for review
Determines that PR is ready for review.
label
Jun 18, 2024
raczu
approved these changes
Jun 19, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
scope: backend
Indicates that PR relates to backend changes.
status: ready for review
Determines that PR is ready for review.
type: feature
PR's categorised as feature.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Create services for database models
Related issue and/or ticket
RSN-14