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

Possible issue with ember-source 3.16.2 #421

Closed
brunoocasali opened this issue Feb 12, 2020 · 6 comments
Closed

Possible issue with ember-source 3.16.2 #421

brunoocasali opened this issue Feb 12, 2020 · 6 comments
Labels

Comments

@brunoocasali
Copy link
Contributor

Hey people! (thanks for the awesome work)

I hate open issues and not pull requests, but this time I must to report...
I have a ember octane version app, and my last upgrade to ember-source 3.16.2 all the infinity scrolls stopped working...

When I reach to bottom, I see the next page with right params but no data are appended to the list and the spinner never stops to spin!

my app versions are:
ember-source 3.16.2 (in 3.16.1 works well)
ember-infinity 2.0.2 (i could not upgrade to the latest because issue #417)
ember-data 3.14

I have a pretty default configuration + a customized .hbs template of infinity-loader.

{{#if hasBlock}}
  {{yield this.infinityModelContent}}
{{else}}
  {{#if this.isDoneLoading}}
    {{#if (and (eq this.infinityModel.length 0) this.infinityModel.reachedInfinity) }}
      <div class="center-column margin-t__24">
        <Empty />
      </div>
    {{/if}}
  {{else}}
    <SpinnerLoading />
  {{/if}}
{{/if}}

Feel free to request me more information, I would love to contribute if I could.

Thanks again!

@snewcomer snewcomer added the bug label Feb 12, 2020
@snewcomer
Copy link
Collaborator

#419

Looks like we do have an issue. Will have to look into this travis CI test failure on 3.16.2...

@GreatWizard
Copy link

I have the same bug behavior and my project i'm still on v1.4.9

@snewcomer
Copy link
Collaborator

My initial inclination is updating an ArrayProxy's content and that not propagating in the template. Still digging....

@snewcomer
Copy link
Collaborator

ref Emberjs issue - emberjs/ember.js#18750

kevinansfield added a commit to TryGhost/Admin that referenced this issue Feb 24, 2020
closes TryGhost/Ghost#11608

- reverts `ember-source` to a working version and temporarily blocks renovate from auto-updating
- ember-infinity issue adopted-ember-addons/ember-infinity#421
- related ember-source issue emberjs/ember.js#18750
@snewcomer
Copy link
Collaborator

snewcomer commented Mar 2, 2020

A fix has been merged and perhaps will be backported to 3.16 or will see it in 3.17!

emberjs/ember.js#18770

@deanylev
Copy link

deanylev commented Apr 28, 2020

Unfortunately this does not seem to be fully fixed. We are facing an issue where the second page of results does not load when scrolling down unless you scroll up and back down again. Does not happen on ember-source 3.16.1, but happens on 3.16.2, 3.16.6, and 3.16.8 from what I've tested.

EDIT: This was actually caused by a couple of observers in our code. It seems the fix for ArrayProxy in 3.16.6 is not equivalent to how it behaved in 3.16.1 and earlier. Worked around it by wrapping their callbacks in Ember.run.next.

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

No branches or pull requests

4 participants