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

Milestone4 - James Torres #227

Closed
wants to merge 30 commits into from
Closed

Conversation

jam-kt
Copy link

@jam-kt jam-kt commented Dec 23, 2023

Developer: James Torres

Closes #168

Pull Request Summary

Added functionality to post comments and send emails using emailJS.

Special Instructions (optional)

Pull Request Checklist

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

Screenshots/Screencast

Screenshot 2023-12-23 060537

jam-kt added 30 commits October 22, 2023 19:06
@jam-kt
Copy link
Author

jam-kt commented Dec 27, 2023

https://bootcamp-project-2023-rouge.vercel.app/
Link to website, I forgot to add into push

@javalosr2004
Copy link
Collaborator

Hi James! Very nice background, some things to consider when working on the projects however is accessibility, as some of the website was difficult to read, such as the comment form which had a black on dark background text for submit. I recommend also adding the "hover: pointer:cursor" style to buttons that are responsive to clicking. Aside from that good job on the website, it is noticeable that you put a lot of effort into it, and all your backend / frontend components seem to work fine! 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