-
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
feat: complete first version #1
Merged
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
…d album controllers with related actions
…include album views
- remove repositories - use dbContext in services - improve ConvertToImageUrl method
- add automapper to album service
- add base validation to album DTOs
- set fixed square size for all image previews - align all album cards when they have diffirent height
- add start page with all albums
- only album album author may edit or delete album and realted elements - edit views to that feature - edit album service to that feature (check and throw related exception)
- create resources folder and related resx files - add localization menu in navigation bar - update DTO validation for localization
- add localization to login/register page - add localization to toastr messages - move localization config logic to extension method
- add new method for home page - edit get method for user album
- add button with ajax that changes depending on the choice - add related service method - add related .resx with localization - update album sort at home page
- update DB entities - update related configuration
- move album DTO to Album folder
- add discussion service with add and get methods - add related views - add related DTO
- add discussion messages create and read methods in service - add related DTO and config them in AutoMapperProfile - add minimal UI - add SignalR to discussions
- add 2 tests for AlbumAdd method with Moq and FluentAssertions -add .yaml file with conf github actions
- remove from solution recently deleted project
- change some common DTOs with properties to records - config automapper profile for AlbumSevice`s GetAlbums methods
- add generic IBaseRepository - add other repositories that inherit IBaseRepository implementation and it own interface - add IUnitOfWork and its implementation
- test several AlbumService methods - test image upload to disk method - use in unit tests different libraries: Bogus, FluentAssertion, Moq, AutoFixture
- change test image file path
- change test image path settings
- setup that image will load to debug folder during build
- copy test image to debug folder during build - add some folders for testing
- add try-catch blocks - recieve toastr messages when error occurred
- messages will check if current user is author and implement related message design - edit Get page and chat.js
- add and setup IdentityServer for login, registration, logout - add mvc project to test identity server in simple way
- add Tech Stack - add Database Scheme
- correct some mistakes
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.
No description provided.