Skip to content

Backend test - Junior Barros #43

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

Closed
wants to merge 33 commits into from
Closed

Backend test - Junior Barros #43

wants to merge 33 commits into from

Conversation

Sylvn001
Copy link

The essentials are ready, there are still a few corrections left, adjustments to the clean code, clearer documentation, validations regarding the monthly investment gain and the example code for sending emails and implementing unit tests.

@Sylvn001
Copy link
Author

Describe: I used the technologies mentioned in the documentation because of their ease and productivity. Nestjs with prism and typescript allow to modularize the files very well and separate the obligations of each module. Although prism is a "recent" ORM, it is very useful and easy to use, I preferred to use it in this application due to the simplicity of the test. Unfortunately, testing with the prism is not my strong point, I've done TDD with typeORM and with mongoose, but never with the prism, which left me a little confused about its development. There are several points that can be improved, tweaks to the code to become cleaner, and other situations, but I tried to do the best possible given my available time and knowledge.

@Sylvn001 Sylvn001 closed this by deleting the head repository Dec 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant