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

Milestone 3B and 4 - Brandon Wong #179

Closed
wants to merge 32 commits into from

Conversation

wbrandon25
Copy link

@wbrandon25 wbrandon25 commented Dec 19, 2023

Developer: Brandon Wong

Closes #134 and #168

Pull Request Summary

  • Blogs are retrieved individually
  • Comments are fetched for each blog
  • Added comment schema

Special Instructions (optional)

Pull Request Checklist

  • Code is neat, readable, and works
  • Comments are appropriate
  • The commit messages follows our guidelines
  • The developer name is specified
  • The summary is completed
  • Ask for a review in communication channels

Screenshots/Screencast

Screenshot 2023-12-19 at 11 19 31 AM Screenshot 2023-12-22 at 11 17 38 PM

@wbrandon25 wbrandon25 changed the title Milestone 3B - Brandon Wong Milestone 3B and 4 - Brandon Wong Dec 23, 2023
@javalosr2004
Copy link
Collaborator

Hi Brandon! Nice work on the project, I liked your implementation of error handling and message displayed in the email form, one area of improvement would be to include a loading screen when waiting for the blogs to load, aside from that your code was easy to read, so good job! Congratulations on finishing bootcamp!

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.

MIlestone 3B
2 participants