Skip to content
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

Platzi Master backend - David Cuy #56

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

DavidCuy
Copy link

@DavidCuy DavidCuy commented Jun 7, 2021

Platzi master backend challenge

Introduction

Hi my name is David, this is my first project using nodejs and express with mongoose.
I'm 28 years old, so I only have one company experience, this because I started a startup 3 years ago. Unfortanaly this startup close a year ago, but I learned a lot of programming techniques.

Challenge resolution

I ran the docker-compose file to up a mongodb database instance, I used this only for development purposes.
I didn't have enough time to make a CI/CD implementation using github actions, so I decided published it in heroku, for a quick implementetion. Here is a documentation using postman.

I used a MVC pattern, with an extra service layer, this because I tried to made a repository pattern at first, but I realized I didn't have enough time. Also I made some endpoints validations, for don't let pass requests that not have a compatible structure.

I made a one little change in get list documents. I decided to implement pagination method, but only if query params passed. So for this change I had to change the test files to run with a expected response.

I needed to change test files, beacuse it had close the connection with the documentdb.

I tried my best with this new language (for me), and always trying to respect structure for make this project bigger.

Thanks!!

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.

2 participants