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

Added Structure for Backend Routing, Middleware, and More Documentation #69

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

SAbbas2018
Copy link
Contributor

[DEV-064] Backend Routes and Middleware

Fixes: #64

  • [Y] Has the code been formatted?
  • [Y] Have dependencies/packages been added?

Changes

Added Backend route structure, middleware skeletons that can be used for authenticating actions. Left TODOS and steps
to complete for functions. Added more documentation to the backend/README.md

Testing

  • [Y] Does the code have sufficient automated test coverage? If not, include a reasoning and steps required to manually QA

@vercel
Copy link

vercel bot commented Jan 13, 2021

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/white-van/dialog/m8w568gye
✅ Preview: https://dialog-git-dev-064.white-van.vercel.app

@jcserv jcserv requested a review from krishchow January 21, 2021 02:18
@krishchow
Copy link
Collaborator

All in all, this looks great, the one note that I have instead of creating structs in the routes, you can directly reference the models. You can also add the relevant json and gin tags there :D

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.

Backend Routes and Middleware
2 participants