-
-
Notifications
You must be signed in to change notification settings - Fork 806
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
Added A Home page section for Twitter mentions #377
Conversation
@Anirban-1490 is attempting to deploy a commit to a Personal Account owned by @reactplay on Vercel. @reactplay first needs to authorize it. |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Anirban-1490 2 comments
plus remove the src/images/group-1825510_1280.jpg file as we do not need it anymore.
src/common/home/Home.jsx
Outdated
@@ -1,4 +1,4 @@ | |||
import { useState, useEffect, useContext } from "react"; | |||
import { useState, useEffect, useContext} from "react"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for correcting it. COuld you please retain the space before the closing } as it was before?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for correcting it. COuld you please retain the space before the closing } as it was before?
ok
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@all-contributors please add @Anirban-1490 for Code |
I've put up a pull request to add @Anirban-1490! 🎉 |
Description
Added a section on the home page that brings the mentions we get for the ReactPlayIO Twitter handle. It's using the Twitter API but instead of dynamically updating it , we just have 10 static tweet ids (twitter only provides tweet upto previous 7 days only) and using react-twitter-widget i am showing this to the website as an embeded.
Fixes #134
Please delete options that are not relevant.
Checklist:
2022.07.14-17.26.mp4
@atapas @koustov sorry, i had to create a new PR as i was not able to push new changes to the old PR. Check this one