Skip to content
This repository was archived by the owner on Apr 28, 2020. It is now read-only.

org.jinstagram.Instagram.getRecentMediaFeed(String userId, int count, String minId, String maxId, Date maxTimeStamp, Date minTimeStamp) not reflective of current API #164

Open
dan-s1 opened this issue May 18, 2016 · 1 comment

Comments

@dan-s1
Copy link

dan-s1 commented May 18, 2016

In the current Instagram API (https://www.instagram.com/developer/endpoints/users/) for the endpoint /users/user-id/media/recent there are no options for using the max_timestamp and the min_timestamp as there was in the old API (https://www.instagram.com/developer/deprecated/endpoints/users/#get_users_media_recent) hence the org.jinstagram.Instagram.getRecentMediaFeed method should reflect this and not allow for arguments Date maxTimeStamp and Date minTimeStamp.

@abargnesi
Copy link
Collaborator

I would suggest we deprecate the API:

/**
 * Get the most recent media published by a user.
 * 
 * @param userId
 * @param count
 * @param minId
 * @param maxId
 * @param maxTimeStamp
 * @param minTimeStamp
 * @return the mediaFeed object
 * @throws InstagramException if any error occurs
 */
 MediaFeed getRecentMediaFeed(String userId, int count, String minId,
     String maxId, Date maxTimeStamp, Date minTimeStamp) throws InstagramException;

and add one that supports count, min_id, and max_id (see /users/{user-id}/media/recent).

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

No branches or pull requests

2 participants