Skip to content
This repository has been archived by the owner on Jul 6, 2019. It is now read-only.

feat(./application): implement input validation and add other fields #14

Merged
merged 16 commits into from
Oct 8, 2018

Conversation

stanleyarmstrong
Copy link
Contributor

Closes #13

@mianuddin
Copy link
Member

Hey @stonewallstan, I know @ohnoitsjmo hasn't reviewed this yet, but heads-up: you have two files in the resume folder that you should probably remove.

…tion-back-end into postgres

* 'master' of https://github.com/slohacks/attendee-application-back-end:
  feat(application): Created question endpoint that returns questions JSON (slohacks#8)
  fix(contributing.md): fix broken image (slohacks#12)
  docs(contributing.md): add contributing guide (slohacks#9)
…application-back-end into input-validation

* 'postgres' of https://github.com/stonewallstan/attendee-application-back-end:
  chore(./resume): delete files for .gitignore to work
  chore(.gitignore): ignore resume files
  feat(application): Created question endpoint that returns questions JSON (slohacks#8)
  fix(contributing.md): fix broken image (slohacks#12)
  docs(requirements.txt): add requirements.txt
  refactor(./attendee_application/settings.py): add environment variables for the postgres database
  feat(all): migrate to PostgresSQL
  docs(contributing.md): add contributing guide (slohacks#9)
@stanleyarmstrong
Copy link
Contributor Author

Closes #16

@mianuddin mianuddin changed the title feat(./application): Input validation feat(./application): implemented input validation and added other fields Oct 8, 2018
@mianuddin mianuddin changed the title feat(./application): implemented input validation and added other fields feat(./application): implement input validation and add other fields Oct 8, 2018
@mianuddin mianuddin self-requested a review October 8, 2018 19:23
Copy link
Member

@mianuddin mianuddin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed during team meeting.

@mianuddin mianuddin merged commit 2073a95 into slohacks:master Oct 8, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants