Skip to content

Conversation

@lpatmo
Copy link
Member

@lpatmo lpatmo commented Sep 13, 2020

What type of PR is this? (check all applicable)

  • ♻️ Refactor
  • ✨ Feature
  • 🐛 Bug Fix
  • 🎨 Enhancement
  • 📝 Documentation Update
  • 🔖 Release
  • 🚩 Other

Context

Lets a user submit a new resource.

Could use help right now:
[ ] Form submission -> 500 error
[ ] checkbox toggle

Screenshots/Recordings (if there are UI changes)

Implementation Details - what was your thought process as you changed the code?

[ ] Can successfully make a POST request against localhost:8000/api/v1/resources
[ ] Form validation
[ ] See a success message after submitting the form
[ ] Tests
[ ] Fix bug re: toggle button

Related Tickets & Documents (Optional)

#55

Filed issue #43 as a next step to do after this PR is merged.

Added tests?

  • 👍 yes
  • 🙅 no, because they aren't needed
  • 🙋 no, because I need help

Added to documentation (readme.md or contributing.md)?

  • 📜 readme.md
  • 📜 contributing.md
  • 🙅 no documentation needed
  • 🙋 I'd like someone to help write documentation, and will file a new issue for it

@lpatmo lpatmo added the do not merge Notify maintainers not to merge a PR (yet) label Sep 13, 2020
@lpatmo lpatmo mentioned this pull request Sep 13, 2020
14 tasks
@lpatmo lpatmo added the help wanted Extra attention is needed label Sep 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do not merge Notify maintainers not to merge a PR (yet) help wanted Extra attention is needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants