Desafio Dev N12 - <Gustavo Molina>#7
Open
gustavomolina wants to merge 29 commits intoaccurate-software:mainfrom
Open
Desafio Dev N12 - <Gustavo Molina>#7gustavomolina wants to merge 29 commits intoaccurate-software:mainfrom
gustavomolina wants to merge 29 commits intoaccurate-software:mainfrom
Conversation
-Controllers: -Added LoginController; -Auth validation -Added LostOrFoundController -GetAllObjects: Get data filtered by OData queries -AddObject: Save AddFoundOrLostViewModel data in the SQL server via business layer -UpdateObject: Receive a UpdateLostOrFoundViewModel data to update data via business layer -Business Layer: -LostOrFoundBusiness -IEnumerable<FoundOrLostViewModel> GetAllLostOrFoundObjects: Calls a vw_GetAllLostOrFoundObjects view list -AddLostOrFoundObject: Calls the sp_AddFoundOrLost stored procedure -UpdateLostOrFoundObject: Calls the sp_UpdateFoundOrLost stored procedure
-Remove ifound-api repository reference
-Added ifound-api repository reference
-It had some updates on the demonstration part of the document
-Created project 'ifound-unittests' in the same solution with ASP.NET Core MVC and XUnit (fix) SetUpDB.sql -Updated by adding the creation scripts for the stored procedures to update and add an object to the database
…hub.io into main Merge in the README.md file
-Created simple project in Angular to ilustrate the project functionality (fix) README.md -Added more descriptions about the application in Front-End
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
1 participant
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.
Ferramentas utilizadas no Back-End:
Ferramentas utilizadas no Front-End:
Padrões de projeto e boas práticas:
Foi implementado o padrão arquitetural MVC seguindo as boas práticas de SOLID. Além disso, foi implementado também o pattern "Unit of Work" para a persistência de dados.
Dificuldades encontradas:
Na tentativa de fazer testes unitários, uma dificuldade foi encontrar a melhor maneira de fazê-los por meio do XUnit. Outra dificuldade foi deixar a pré-instalação do projeto o mais intuitiva possível. Para isso, criei também um script em SQL para construir todas as estruturas no banco de dados.
O que seria possível melhorar:
Seria possível implementar uma cobertura maior de testes unitários e ampliar as funcionalidades no front-end para o usuário poder gerar também relatórios, assim como é feito via requisições HTTP.