STD-26 | TheoryBlock Repository#16
Merged
Merged
Conversation
Trying to fix it now
I`ll try to fix it now
WerewolfNet
reviewed
Sep 12, 2021
_context is now readonly if statement and FindAsync in GetTheoryBlocksByCourseIdAsync was deleted "potentially deleted if statement" in Update was added
"potentially deleted if statement" comment was removed
"()" in return new OperationResult were removed too
WerewolfNet
reviewed
Sep 13, 2021
AlternativeResult was renamed to BuildErrorResponse Method moved to the end of the TheoryBlockRepository.cs
WerewolfNet
approved these changes
Sep 14, 2021
WerewolfNet
pushed a commit
that referenced
this pull request
Sep 30, 2021
* STD-26 | TheoryBlock Repository (#16) * ITheoryBlockRepository was added * TheoryBlockRepo was added Trying to fix it now * TheoryBlockRepository.cs was added I`ll try to fix it now * TheoryBlockRepository.cs done * TheoryBlockRepository.cs redacted, pre-dev 1 _context is now readonly if statement and FindAsync in GetTheoryBlocksByCourseIdAsync was deleted "potentially deleted if statement" in Update was added * Minor changes, pre-dev 2 "potentially deleted if statement" comment was removed * SaveChangesAsync was added, pre-dev 3 "()" in return new OperationResult were removed too * Method AlternativeResult was added to implement DRY principle, pre-dev 4 * AlternativeResult method changes, pre-dev 5 AlternativeResult was renamed to BuildErrorResponse Method moved to the end of the TheoryBlockRepository.cs * STD-24 | Tag Repository (#17) * Added ITagRepository interface and TagRepository class * Were realised methods in TagRepository * Inverted if statements. Co-authored-by: Егор Павшук <egor.pavshuk@outlook.com> Co-authored-by: Maria Dodonova <maridodonova@gmail.com> * Std 27 (#25) * STD-14 | Add code analyzers (#18) * STD-29 | Add global error handler (#19) * STD-29 | Add global error handler * Fixes * Created IPracticalBlockRepository interface in the StudySharp.Domain/Repositories folder Created PracticalBlockRepository class in the StudySharp.DomainServices/Repositories folder * Fixed StudySharpDbContext.cs * PracticalBlockRepository.cs sealed added Co-authored-by: SleepyF0X <kotohomka@gmail.com> * Std 25 (#23) * STD-14 | Add code analyzers (#18) * STD-29 | Add global error handler (#19) * STD-29 | Add global error handler * Fixes * Create build.yml * Update build.yml * Update build.yml * Update build.yml * Update main.yml * Update main.yml * STD-5 | CI\CD (#20) * Docker support * Update main.yml * Update main.yml * Update main.yml * Update main.yml * Update main.yml * Update main.yml * Update main.yml * Update main.yml * Update Dockerfile * Docker support * Test * Test db * Test db v2 * Update main.yml * Test something * Test dev mode * Test dev mode v2 * Test dev mode v3 * Test dev mode v3 * Update main.yml * Update main.yml * Added ICourseRepository interface and CourseRepository class * STD-25 | Added ICourseRepository interface and CourseRepository class Co-authored-by: SleepyF0X <kotohomka@gmail.com> * STD-36 | PracticalBlock commands\queries * Fixed STD-36 * Fixed spaces * Some fixes * Rename and Cancellation token * Fixed Update * Rename handler Co-authored-by: zefirlover <71660586+zefirlover@users.noreply.github.com> Co-authored-by: Egor-Pavshuk <70218372+Egor-Pavshuk@users.noreply.github.com> Co-authored-by: Егор Павшук <egor.pavshuk@outlook.com> Co-authored-by: Maria Dodonova <maridodonova@gmail.com> Co-authored-by: Andrew <47719935+Evenya@users.noreply.github.com> Co-authored-by: SleepyF0X <kotohomka@gmail.com>
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
No description provided.