Skip to content
This repository has been archived by the owner on Feb 22, 2022. It is now read-only.

Adds swagger #92

Merged
merged 10 commits into from
Oct 26, 2019
Merged

Adds swagger #92

merged 10 commits into from
Oct 26, 2019

Conversation

frankcash
Copy link
Owner

@frankcash frankcash commented Oct 26, 2019

Related Issue

#91

Changes Proposed

  • Update Postgres to Postgres 12-alpine
  • Adds container name to hackerqueue pod
  • Adds swagger to project
  • Adds wrapper util for ensuring response is properly wrapper and unit tests for it

Caveats

The base level of this project is getting kind of huge now

Images

Swagger UI on /api-docs/
Screen Shot 2019-10-26 at 11 49 45 AM

New API Response
Screen Shot 2019-10-26 at 12 08 47 PM

UI working with new model
Screen Shot 2019-10-26 at 12 08 36 PM

New test passing
Screen Shot 2019-10-26 at 12 29 48 PM

@frankcash frankcash self-assigned this Oct 26, 2019
@frankcash frankcash marked this pull request as ready for review October 26, 2019 19:15
@frankcash frankcash merged commit 07a5de2 into master Oct 26, 2019
@frankcash frankcash deleted the frankcash/91 branch October 26, 2019 19:36
@frankcash frankcash mentioned this pull request Oct 26, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant