Skip to content

Conversation

@reenphygeorge
Copy link

Summary of changes I made

  1. Containerized the entire code base. Now it can be started by just one command - docker compose up
  2. Changed the structure of project minimizing repetition and added global connection support. (router - handler based structure)
  3. Added a file based logger for better log collection (good for production)
  4. Changed to env file instead of hard coding credentials
  5. Proper error handling is done
  6. Code is commented at necessary parts

Suggestions

  1. Instead of using raw query, an ORM (like Prisma or GORM) would be better as it is more safe for production
  2. Swagger docs would be better as it would provide a good api docs (helpful for frontend team)

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