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

529 create post endpoint #534

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

bzzz-coding
Copy link
Member

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

@LoTerence
Copy link
Member

LoTerence commented Oct 8, 2024

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 😕

@bzzz-coding
Copy link
Member Author

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?

@LoTerence
Copy link
Member

LoTerence commented Oct 10, 2024

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.

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.

2 participants