-
-
Notifications
You must be signed in to change notification settings - Fork 131
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
Document/allow usage without ember-data #57
Comments
Hi @treyhunner ! It's going to be very possible after #32 is merged, as we'll give the addon user a way to pass in the promise generator for the infinityRoute. We've also just exposed a hook ( |
yup - that should be fine. Gonna need to update #32 to be a currying style overridable "promise generator", so I'll be a sec before it gets in |
+1, would be great to have a way to use this without Ember Data. |
Looks like #32 merged? Still unclear from the README.md that Ember Data is required or not. The |
Thanks @john-kurkowski ! Ember Data is still a requirement. Given your work on this already - do you want to take a stab at completely de-coupling it? |
Sure. You can see what I described in my allow-use-without-ember-data branch. Before I invest in a full blown PR, is it roughly what you were looking for? |
Hi @john-kurkowski ! I'm just seeing this now - sorry! I watch a ton of repos and occasionally miss stuff. This is exactly what I was thinking. Would you be able to make a PR for us to discuss this further? |
I don't have time now, but I've set a reminder to look at this again in 1 month. I may have time then. If you can't wait, do with my branch what you will! |
Thanks @john-kurkowski ! |
Don't thank me yet. Things haven't let up for me. I unfortunately won't be getting back to this any time soon. I'll leave my fork's branch available for the intrepid. |
thanks @john-kurkowski one of these days! |
Any updates on supporting infinity scroll w/o ED. |
hi @xomaczar - no, not yet. It most likely won't be done until someone submits a PR, or I need the feature myself. If you'd like to take a stab at implementing @john-kurkowski's approach - I'd be down to accept that PR! |
@hhff I did an initial spike, it looks promissing, however the only problem is related to ember-version-is addon which depends on ember-data module - specifically its VERSION attr. As you might imaging this creates problems for resolver when it tries to find ember-data in the consuming app. I was able to fix it in my app by providing an ember-data fake module - but it's a hack and not fit for general use. |
I haven't pushed the code to my fork yet as I was trying to figure out a nicer way to handle it. |
@hhff any ideas on removing/patching ember-version-is dependency on ember-data? |
Hi @xomaczar - should be super straightforward to patch Failing that - I'm not against removing ember-version-is completely - we don't necessarily need it to do simple version checking. Thoughts? |
I am ok with either one. Although, the second option results in less code being bundled in. |
yup - I'd be cool with the second @xomaczar ! I'm sure there's some libs out there that are already doing something like that we could jack |
Hi - I was wondering what the status was regarding removing or patching |
hi @tyreepace - at the moment I'm focused on removing the coupling of Ember Infinity pagination control to the route. IMO - the workload for Non-Ember Data is:
If you'd like to take a stab at removing Ember Version is - please do! |
@tyreepace @xomaczar Hey all! Just if anybody is curious, you can use your own custom data store now in the options hash. Another note, the query method you use needs to return a promise, which I think in most cases makes sense, but will remove that req. |
What API should the custom store expose? I see you using push and findAll - anything else?
… On Apr 18, 2017, at 10:37 AM, Scott Newcomer ***@***.***> wrote:
@tyreepace @xomaczar Hey all! Just if anybody is curious, you can use your own custom data store now in the options hash. Another note, the query method you use needs to return a promise, which I think in most cases makes sense, but will remove that req.
#215
https://github.com/hhff/ember-infinity/blob/master/tests/acceptance/infinity-custom-store-test.js#L8
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.
|
@xomaczar you can define it with |
Can this be used without ember-data?
I am currently using ember-data without using adapters/fetching. I plan to remove ember-data from my app soon. Will either of these two scenarios work with ember-infinity?
The text was updated successfully, but these errors were encountered: