[RSN-37] - Created services #42
Merged
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
bilimig
added
the
status: ready for review
wzarek
requested changes
Jun 6, 2024
There was a problem hiding this comment.
Okay so:
- format
- do mappers
- throw errors
@raczu please check if I did not miss anything. Don't know what about interest/tags/parameter since we only create it on assigning to some event/user and delete only with detaching from event/user, am I right?
wzarek
requested changes
Jun 10, 2024
Server/ReasnAPI/ReasnAPI.Tests/Services/InterestServiceTests.cs
Outdated
Show resolved
Hide resolved
Server/ReasnAPI/ReasnAPI.Tests/Services/ParameterServiceTests.cs
Outdated
Show resolved
Hide resolved
|
@bilimig Resolve conflicts |
Also added geting all elements and by filter(not done properly yet)
… and Tag Services
Checked if any of Parameters, Tags or Statuses are used before delating/updating.
Changed delete methods to bool, updated mocks on tests.
Changed using bridge tables to using colections in EF.
added a logic for parameters in event
wzarek
approved these changes
Jun 12, 2024
bilimig
commented
Jun 13, 2024
raczu
requested changes
Jun 13, 2024
wzarek
added
status: require changes
wzarek
requested changes
Jun 13, 2024
bilimig
added
status: ready for review
wzarek
requested changes
Jun 14, 2024
wzarek
approved these changes
Jun 17, 2024
raczu
approved these changes
Jun 17, 2024
mkoper02
added a commit
that referenced
this pull request
Jun 18, 2024
* 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 a commit
that referenced
this pull request
Jun 19, 2024
* Created services classes and added TODO methods for each class * [RSN-38] - Create services (address, comment, objecttype, participant, 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 * [RSN-37] - Created services for Event Tag Parameter Interest Image (#42) * 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]> * Update Entity Framework version * Changed exceptions location * ReasnContext update --------- Co-authored-by: Owczarek Kamil <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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
Created CRUD services for:
TagService
ParameterService
IntrestService
ImageService
EventService
Created Tests for
TagService
ParameterService
IntrestService
ImageService
Related issue and/or ticket
RSN-37