Skip to content

wiki module:added images in case of Disambiguation #265

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

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

wiki module:added images in case of Disambiguation #265

wants to merge 5 commits into from

Conversation

avirlrma
Copy link

@avirlrma avirlrma commented Aug 9, 2017

No description provided.

@avirlrma
Copy link
Author

Any Updates?

@@ -40,7 +39,7 @@ def process(input, entities):
}
}
buttons.add_postback('Wikipedia Summary', payload)
template.add_element(title=data.title, item_url=data.url, image_url=image_url,
template.add_element(title=data.title, item_url=data.url, image_url=data.images[0],
Copy link
Owner

Choose a reason for hiding this comment

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

The images list might be empty. In that case, let's default to the current image_url.

@avirlrma
Copy link
Author

@swapagarwal Please review sir

@swapagarwal
Copy link
Owner

Hey @aviral1701, I deployed this change to test it; but the output isn't that good. What do you think?

@avirlrma
Copy link
Author

@swapagarwal Ya result is not so good.May be we should stick with the wiki image only.

@swapagarwal
Copy link
Owner

Can we get the main wiki image using some other API? If not, let's drop this.

@avirlrma
Copy link
Author

@swapagarwal Hey, sorry for late response,was busy with my midterms.
So I was thinking once we get the page IDs,we can make a simple request to the wikipedia api(Second link you mentioned) to fetch the main image.I have tested it and it works fine.

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

Successfully merging this pull request may close these issues.

2 participants