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

Fix bad async suggestions rendering logic #1200

Merged
merged 1 commit into from
Apr 29, 2015

Conversation

jharding
Copy link
Contributor

No description provided.

@jharding jharding added this to the 0.11.2 milestone Apr 28, 2015
jharding added a commit that referenced this pull request Apr 29, 2015
Fix bad async suggestions rendering logic
@jharding jharding merged commit 312d481 into twitter:integration-0.11.2 Apr 29, 2015
@mayank-saavn
Copy link

This fixes it. Thanks.

@jarthod
Copy link

jarthod commented May 26, 2015

#1212 also fixes but with tests ;)

@PacciniBruno
Copy link

Same here :'(

@ultimatemanu
Copy link

Same for us :) Spent also a few hours to find where the problem was.

@j-kwon
Copy link

j-kwon commented Jun 10, 2015

same here, look forward to merging into the master asap.

@gremo
Copy link

gremo commented Jun 16, 2015

#1200 works for you guys? Because for me it doesn't. Limit: 5, result set: 5, shown: 0. #1212 fixes the problem at least for me.

@eins78
Copy link

eins78 commented Jun 22, 2015

I can confirm that pr #1200 (this) does not fix issue #1232, but pr #1212 does (and, as others have pointed out comes with a test).

@Floby
Copy link

Floby commented Jul 6, 2015

Any idea when this is going into master ?

@davis
Copy link

davis commented Jul 6, 2015

ah i finally found why this is happening :D

@terrasoff
Copy link

not working... forced to switch to fork #1212

cristian-sima pushed a commit to cristian-sima/Wisply that referenced this pull request Oct 23, 2015
@isdampe
Copy link

isdampe commented Dec 20, 2016

+1 for merge...

@danielgasser
Copy link

Still not in master? How do I fix that and rebundle all files?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.