CivicTechJobs
CivicTechJobs copied to clipboard
hackforla
529 create post endpoint
Fixes #529
Changes
- Create Post serializer
- Created CreatePost view
- Configured URL path
/api/create_post - Tested POST request with Postman, and confirmed that new JSON data is added to the Post table in the database
- Added unit test for CreatePost, but the test currently is not working
Hey @bzzz-coding can you fix the merge conflicts here when you have a chance? Thanks!
Edit: Wait a second, was this code branched from the old backend? If so we might have to redo it entirely 😕
Hey @bzzz-coding can you fix the merge conflicts here when you have a chance? Thanks!
Edit: Wait a second, was this code branched from the old backend? If so we might have to redo it entirely 😕
Hey @LoTerence, yes this was old, but I can redo it and submit a new PR if you want to close this one, or should i wait till we have a better understanding of the user flow?
Don't redo it yet - I want a better understanding of the user flow, and using that I think we may have to change the data model.
Right now we have an Opportunities model, and I'm not really understanding the difference between that and Post here.
Let's pause on this PR and keep your work here for future reference.
We have a POST endpoint for opportunities now (#623 ). It's just missing the unit test, which we can add later. So I'm closing this PR. Thanks for your work Bitian!