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

[#2PlaysAMonth]: Dad Jokes #942

Merged
merged 14 commits into from
Feb 10, 2023
Merged

[#2PlaysAMonth]: Dad Jokes #942

merged 14 commits into from
Feb 10, 2023

Conversation

Bobby-coder
Copy link
Contributor

Dad Jokes

Dad Jokes is a fully functional react app which offers the various categories of jokes to enjoy.

Fixes #930

Type of change

  • New feature (non-breaking change which adds functionality)

How Has This Been Tested?

It is tested locally.

Checklist:

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • My changes generate no new warnings

@vercel
Copy link

vercel bot commented Feb 6, 2023

@Bobby-coder is attempting to deploy a commit to a Personal Account owned by @reactplay on Vercel.

@reactplay first needs to authorize it.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Hello! Thank you for your contribution 😊.
Please reference the issue number in the description if you are fixing a bug.
If you are implementing a feature request, please check with the maintainers that the feature will be accepted first.
Stale Marking : After 30 days of inactivity this PR will be marked as stale PR and it will be closed and locked in 7 days if no further activity occurs.

@vercel
Copy link

vercel bot commented Feb 6, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated
react-play ❌ Failed (Inspect) Feb 10, 2023 at 7:24AM (UTC)

@atapas atapas changed the title Dad Jokes [#2PlaysAMonth]: Dad Jokes Feb 6, 2023
Copy link
Member

@atapas atapas left a comment

Choose a reason for hiding this comment

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

You have put a great effort on this one @Bobby-coder Good Job.

Please take care of the review comments. Also try reducing the size of the cover image. Its more than 4MB now.. Try making it in KBS for faster loading in a slow network.

I'll merge this one sooner you address all the comments. Good Luck!

src/plays/dad-jokes/DadJokes.js Outdated Show resolved Hide resolved
src/plays/dad-jokes/Readme.md Show resolved Hide resolved
src/plays/dad-jokes/components/jokeList/jokeList.css Outdated Show resolved Hide resolved
src/plays/dad-jokes/styles.css Show resolved Hide resolved
@atapas
Copy link
Member

atapas commented Feb 10, 2023

@Bobby-coder Is it ready for a final review? Have you taken care of all the comments?

@Bobby-coder
Copy link
Contributor Author

@Bobby-coder Is it ready for a final review? Have you taken care of all the comments?

Yes it is ready for the final review

Copy link
Member

@atapas atapas left a comment

Choose a reason for hiding this comment

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

@Bobby-coder Great Job. LGTM.

Do not forget to share it with the world.. and if you do so, please tag #2PlaysAMonth and ReactPlayIO. This play will be available on the ReactPlay in 10 minutes after a successful build.

Looking forward to the second one!

@Bobby-coder
Copy link
Contributor Author

@Bobby-coder Great Job. LGTM.

Do not forget to share it with the world.. and if you do so, please tag #2PlaysAMonth and ReactPlayIO. This play will be available on the ReactPlay in 10 minutes after a successful build.

Looking forward to the second one!

Sure 😊

@atapas atapas merged commit f31b8a2 into reactplay:main Feb 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[#2PlaysAMonth]: Dad Jokes
2 participants