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

Add Authors API and update Author API #80

Merged
merged 6 commits into from
Oct 29, 2023

Conversation

kaynzhel
Copy link
Contributor

  • based on project reqs (link is on this repo's readme)
  • Added views, serializers, and then tested them
  • Had to update Follow, Follower, and Inbox tests/serializer since they also need to serialize a given author. Since the AuthorSerializer is updated, we need to update how we're handling it here as well.
  • Updated the author_id check in PostsList component with the updated id field (it was initially just uuid, now id returns an id url)
  • NOTE: I didn't do post method for the AuthorView as to not interfere with current user story As an author, I want to be able to use my web-browser to manage my profile #24

TEST

  • Feel free to test the web app to QA

POSTMAN

Single Author Authors
Screenshot 2023-10-27 at 8 40 18 PM Screenshot 2023-10-27 at 8 40 10 PM

Copy link
Contributor

@rmgutierrez rmgutierrez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm!

@rmgutierrez rmgutierrez merged commit 77980ac into main Oct 29, 2023
@rmgutierrez rmgutierrez deleted the add-authors-and-single-author-api branch October 29, 2023 05:51
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