-
Notifications
You must be signed in to change notification settings - Fork 858
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
Add a way to get carousel images #650
Comments
Thanks for raising this feature @stevenschobert. May I ask for an estimated ETD? |
Afraid I can't give one @pfvqueiros, sorry! I can only work on this project during my free time, which is limited and unpredictable. (For example, I was only able to rewrite the v2 of this project because I finished a previous project early, and sacrificed some sleep.) If you're needing it urgently for a client project, you might have to implement it yourself using a callback options that are provided (like |
Totally understand. Thanks again for your great work :) |
Just a note on this after some initial investigation - the extra images in a carousel are available as an array of ID's - which means that they require 1 request per image. If you have a lot of carousels with a lot of images in them, this would eat up your API request allowance a lot faster (the default allowance is 200/hour, and normally instafeed.js uses 1 per user for the initial load). We might need to look at a scheme where the extra images are loaded on demand (the user clicks 'next' to trigger the carousel to load), rather than everything automatically. |
I believe if you add "children{media_url,thumbnail_url}" to the call out to the graph api, that returns all children of carousel albums. At least it does in the (limited) tests that I've done.
|
Any luck on this? I am looking to display album images also on my website. |
This issue has been automatically marked as stale because it hasn't had new comments in the last 3 months. It will be closed if no further activity occurs. If you still need assistance with this issue, or believe it shouldn't be closed, please respond with a new comment to let us know. |
I changed the url variable as @djqdos suggested. Then added lightSlider to my project. Find my code in attachment ;-) |
This issue has been automatically marked as stale because it hasn't had new comments in the last 3 months. It will be closed if no further activity occurs. If you still need assistance with this issue, or believe it shouldn't be closed, please respond with a new comment to let us know. |
Feature request to add a way to get the images from a carousel album
The text was updated successfully, but these errors were encountered: