-
Notifications
You must be signed in to change notification settings - Fork 0
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
For dummy events, minimum of 3 events? #68
Comments
PR #67 is attempting to resolve this:
The other bottom-carousel endpoints I see are "around me" and "by venue." I'll look into those soon. Are there others that I'm forgetting? |
Hey there, Liam! Those are the four footer slider endpoints, you haven't missed any. :) Thanks! |
I added dummies to the other two endpoints as well-- this is now merged and deployed. The hyperlink is still active for the dummy events, but otherwise I think it looks right on this end! |
Hi, Liam!
As we're aiming to display a minimum of 3 events in the bottom carousel, can you pad the API results so that the API returns 0 or 3+ events? If only 1 event is returned, we'd need 2 dummy items; If 2 events come back, we'd need 1 dummy item.
Thanks!
Mark
/cc @desigonz
The text was updated successfully, but these errors were encountered: