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

Adds support for Instagram API v1 and v2 mappers #105

Merged
merged 1 commit into from
May 15, 2020

Conversation

whiteshadow-gr
Copy link
Contributor

@whiteshadow-gr whiteshadow-gr commented May 7, 2020

Added support for v1 and v2 of the Instagram API items.

Adds more unit tests regarding instagram mappers

@whiteshadow-gr whiteshadow-gr force-pushed the ds-303-instagram-mapper branch from a4f4202 to 08a8ce8 Compare May 7, 2020 19:16
@whiteshadow-gr whiteshadow-gr changed the title Adds support for v1 and v2 mappers Adds support for Instagram API v1 and v2 mappers May 7, 2020
@whiteshadow-gr whiteshadow-gr linked an issue May 7, 2020 that may be closed by this pull request
Copy link
Contributor

@augustinas augustinas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Update the mappers according to the changes requested for the Instagram plugin PR#48

@whiteshadow-gr whiteshadow-gr force-pushed the ds-303-instagram-mapper branch from 08a8ce8 to cedd691 Compare May 11, 2020 12:33
@whiteshadow-gr whiteshadow-gr requested a review from augustinas May 11, 2020 12:33
@whiteshadow-gr whiteshadow-gr force-pushed the ds-303-instagram-mapper branch from cedd691 to 700e9ae Compare May 15, 2020 08:41
@whiteshadow-gr whiteshadow-gr force-pushed the ds-303-instagram-mapper branch from 700e9ae to 62400a7 Compare May 15, 2020 08:50
List(EndpointQuery("instagram/feed", None,
unixDateFilter.map(f ⇒ Seq(EndpointQueryFilter("ds_created_time", None, f))), None)), Some("ds_created_time"), Some("descending"), None)

Seq(propertyQueryv2, propertyQueryv1) //ordering is on purpose
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what does this ordering impact?

Copy link
Contributor

@augustinas augustinas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@whiteshadow-gr whiteshadow-gr merged commit 445fbec into dev May 15, 2020
@whiteshadow-gr whiteshadow-gr deleted the ds-303-instagram-mapper branch May 15, 2020 11:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Update feed mapper for instagram data
2 participants