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

Infinite scroll broken w/ ember-in-viewport 3.5.0+ #69

Closed
nickschot opened this issue May 7, 2019 · 6 comments · Fixed by #73
Closed

Infinite scroll broken w/ ember-in-viewport 3.5.0+ #69

nickschot opened this issue May 7, 2019 · 6 comments · Fixed by #73
Labels
bug Something isn't working

Comments

@nickschot
Copy link
Owner

Current workaround: set a resolution for ember-in-viewport 3.4.0.

@nickschot nickschot added the bug Something isn't working label May 7, 2019
@nickschot
Copy link
Owner Author

Did a patch release to work around this issue for now. See #70

@amauryt
Copy link

amauryt commented Sep 27, 2019

Hi Nick. Thank you for releasing this add-on and the related companion for ember-bootstrap.

I just wanted to give you a heads up and also to help other people that may have the same problem I had when upgrading to ember 3.11, as my model-selects stopped working with the following error:

Assertion Failed: You must call `this._super(...arguments);` when overriding `init` on a framework object. Please update <suoniamo-frontend@service:-observer-admin::ember463> to call `this._super(...arguments);` from `init`.

The origin is within ember-in-viewport and it is a known issue in older releases of that add-on. However, I noticed that in #70 you locked the version of ember-infinity(and thus of ember-in-viewport) as a workaround for issue #69.

For the moment my workaround to is to completely disable infiniteScroll by setting it to false in model-selects options. Fortunately, for my application infinite scrolling is not absolutely necessary. Nonetheless, I think this that this workaround is far from ideal because the locked dependency is included even if it is not used, and it may cause conflicts with other add-ons or apps that use newer versions of ember or of the other add-ons.

@nickschot
Copy link
Owner Author

Thanks for the thorough heads up! I'll take a look to see if I can find the issue with ember-infinity (or if it might be resolved now).

@nickschot
Copy link
Owner Author

nickschot commented Sep 27, 2019

@amauryt seems this issue is solved in newer ember-in-viewport versions. I'm preparing an update!

My guess is this fixed it: DockYard/ember-in-viewport#196

nickschot added a commit that referenced this issue Sep 27, 2019
* update to ember-cli 3.12 & update all dependencies

* fix sinon spy usage
@nickschot
Copy link
Owner Author

@amauryt should be solved in 0.2.0!

@amauryt
Copy link

amauryt commented Sep 30, 2019

@amauryt should be solved in 0.2.0!

That was a quick response, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants