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 4 - Aarav Sharma #213

Closed
wants to merge 43 commits into from
Closed

Conversation

aarav27
Copy link

@aarav27 aarav27 commented Dec 23, 2023

Developer: Aarav Sharma

Closes #168

Pull Request Summary

  • Added POST API endpoints to create comments in blog and project pages
  • Created functionality to sends emails to myself through contact me form
  • Deployed website with Vercel

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-20 at 10 30 07 PM Screenshot 2023-12-22 at 8 41 44 PM Screenshot 2023-12-22 at 8 42 35 PM

@javalosr2004
Copy link
Collaborator

Hi Aarav! Nice work on your website, I did get rick rolled though :(. Some areas of improvements to look at when working on the actual project would be to only show the cursor:pointer option when you can actually click on a button, otherwise it will lead to a frustrating user experience, ie. your blog links/project links are only clickable via the read more link. The separation of resume components was nice, alongside the good backend error handling with sending out emails. 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 4
2 participants