Skip to content

[Draft] [Issue-55] Submit a resource #154

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

Closed
wants to merge 3 commits into from
Closed

[Draft] [Issue-55] Submit a resource #154

wants to merge 3 commits into from

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.

3 participants