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

Pokedex POST assignment - Shane #226

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

ReadyHash
Copy link

@ReadyHash ReadyHash commented Apr 8, 2020

Deliverable Submission

Please describe your comfort and completeness levels before submitting.

Comfort Level (1-5): 1

Completeness Level (1-5):1

How many hours did you spend on this assignment?
3 hours

What part did you spend the most time on?
Trying to make POST work with views

Does the assignment run without errors?
No

If the assignment has errors, what are they?
Cannot POST /newpokedex.json(a new empty json file)

Describe one part of the code or choose one line you would improve upon.
What would you improve?

What did you think of this deliverable? Any other comments?
Not very confident in POSTing datas, too much setup to remember

**Deliverable Submission 2**

Please describe your comfort and completeness levels before submitting.

Comfort Level (1-5): 1

Completeness Level (1-5):1

How many hours did you spend on this assignment?
5 hours

What part did you spend the most time on?
Trying to get POST to work, and getting array within pokedex object

Does the assignment run without errors?
No

If the assignment has errors, what are they?
Cannot POST /newpokedex.json(a new empty json file)

Describe one part of the code or choose one line you would improve upon.
What would you improve?
-
What did you think of this deliverable? Any other comments?
Still unable to POST, alittle confused around objects within arrays or arrays within objects

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.

1 participant