Dev06 08 ricardo collado practice01#2
Open
RicardoColladoRothschild wants to merge 2 commits intoorodriguez:mainfrom
Open
Dev06 08 ricardo collado practice01#2RicardoColladoRothschild wants to merge 2 commits intoorodriguez:mainfrom
RicardoColladoRothschild wants to merge 2 commits intoorodriguez:mainfrom
Conversation
…the one that control the objects that came in the request and the ExpenseController transform from a JSON to Expense object. By simple adding a property to the class, now it can recieve the Description property within the JSON when requested
…modified some clases and some new methods to some interfaces and then provide behaviors for these methods. Method that were added: GetById, Remove, DeleteExpense. Provide logic for these method and the controller coudl take care of the Request that comes from the client
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.
Add a Description property to Expense class: Expense class object is the one that control the objects that came in the request and the ExpenseController transform from a JSON to Expense object. By simple adding a property to the class, now it can recieve the Description property within the JSON when requested
Create a Delete endpoint: In order to create a Delete endpoint, must modified some clases and some new methods to some interfaces and then provide behaviors for these methods. Method that were added: GetById, Remove, DeleteExpense. Provide logic for these method and the controller coudl take care of the Request that comes from the client
Additional": Leave some comments on the code that helped me to understand what was actually happen on the solution.