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

Week 7: Happy Thoughts Viktoria O. #63

Open
wants to merge 14 commits into
base: main
Choose a base branch
from

Conversation

viktoria-olo
Copy link

Netlify link

https://flourishing-mooncake-e332de.netlify.app/

Collaborators

Individual project

const newMessage = event.target.value;
setMessage(newMessage);
setCharactersUsed(newMessage.length);
if (newMessage.length > 140) {
Copy link

Choose a reason for hiding this comment

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

nice that you added a stretch goal!

Copy link

@jyy009 jyy009 left a comment

Choose a reason for hiding this comment

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

Hi! Your app looks just like the design and meets all the requirements and accessibility. Your code is very organized and easy to follow your thought process which makes it nice to read and review. I thought the number of components you have and the way it's mounted together was very clean! Nice job!

Copy link
Contributor

@dzc1 dzc1 left a comment

Choose a reason for hiding this comment

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

Hey Viktoria great work! 💪💪

Loved to see the distribution off Data through the Thoughts.jsx file, simple yet effective. What can I say but welcome to modern web development and II hope that you are enjoying the React framework. From my end it's seems that your grasping of the useEffect hook is pretty clear and you also implemented the date-fns lib. Great approach.

Keep up the great work!

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.

3 participants