-
-
Notifications
You must be signed in to change notification settings - Fork 692
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
fix: tweetLimit in community/newsroom isn't been reflected #3138
fix: tweetLimit in community/newsroom isn't been reflected #3138
Conversation
✅ Deploy Preview for asyncapi-website ready!Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site configuration. |
⚡️ Lighthouse report for the changes in this PR:
Lighthouse ran on https://deploy-preview-3138--asyncapi-website.netlify.app/ |
Do we have any fix for this @Saumya40-codes |
I dont think so, untill we use official twitter apis. I'll still lookout if any other alternatives still exists |
@Saumya40-codes Instead of providing strict height to that element, can't we restrict the embed to only fetch 2 tweets there? |
I tried looking for all possible alternatives, but none really were able to work as intended |
Close with me |
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.
Can you help me?? Thank pro
@Saumya40-codes what's the status of this issue? |
|
@Saumya40-codes I can't see the tweets even though I'm logged in to twitter |
Twitter plugins needs to be inconsistent enough then imo here im using their official site for the same ( one from browser having not logged in to twitter and otherwise) here is the link for the same: https://publish.twitter.com/?dnt=1&query=%40AsyncAPISpec&widget=Timeline Additionally, this issue is pretty much from twitter plugins only, ref this and this |
@Saumya40-codes any update on this issue? |
Nope the ones I stated here looks only to be probable fixes: Rest I'll be needing some suggestions on those for proceeding |
@derberg I think we need your assistance here |
Sorry @Saumya40-codes we have moved this issue under bounty program so @akshatnema will be working on this one. |
Description
In community/newsroom page here the amount of tweets which are being loaded to show the timeline is uncontrolled, even though we've mentioned
tweetLimit:2
within our code, it turns out that this has been the issue with the main package/library itself and nothing can be done much to solve it.Current issue:
AsyncAPI.Newsroom._.AsyncAPI.Initiative.for.event-driven.APIs.-.Google.Chrome.2024-08-07.20-50-46.mp4
To fix this issue though we can provide a specific height for this particular component and enable scrolling for the rest of the tweets to be seen:
AsyncAPI.Newsroom._.AsyncAPI.Initiative.for.event-driven.APIs.-.Google.Chrome.2024-08-09.22-05-18.mp4
Related issue(s)
Resolves #3132
Additionally, the problem of user who isn't logged in to twitter seeing a message something like below as pointed out by @sambhavgupta0705
Unfortunately this issue exists in twitter's core widget plugin service only ( ref this )