Processed Issues19

This page shows a list of issues for today.com that were approved or declined by our admins.

Template #1 Issue #1
May 20, 2017
This article page should be generated
Nizar
I just fixed the issue :D thanks for pointing it out! I still don't know how to update the preview though... D:
Accepted by admin
To update your template, you would need to resubmit your template to the contest.
Template #1 Issue #2
May 21, 2017
"share tweet pin email" is not necessary
Nizar
I just fixed the issue, I just don't know how to reupdate the thing. Thanks for pointing it out :)
Accepted by admin
To update your template, you would need to resubmit your template to the contest.
Template #3 Issue #1
May 28, 2017
Why?
Declined by admin
Issues for pages with unsupported content may be considered if a link is provided to a template that supports this content (including the video).
Template #3 Issue #2
May 30, 2017
Missing caption and credits
Accepted by admin
Template #4 Issue #1
Jun 6, 2017
1.1. Pages with static article-like content (documentation)
Declined by admin
It's optional to generate IV for this page.
Template #4 Issue #2
Jun 6, 2017
Videos doesn't work. Shouldn't it be made into <iframe> (it's possible) according to checklist 6.8.2.?
chuchu
Omg, bugged json..
Accepted by admin
Template #5 Issue #1
Jun 6, 2017
This related video can be deleted. Also it's their own embed. On /today.com/ it can be reworked. On /*.today.com/ nope. No point to make iframe of their own embeds if u can get source
Sergey Popkov
Why delete this video? It can really relate to post. In 6.8.2. checklist it's said (Sounds like i'm citing Bible :) ) that first of all you should try to put unsupported player into iframe.
Declined by admin
Issues regarding articles with unsupported content may be considered if they include a link to a template that handles the disputed content well.

Also, please note that "related video" content is optional, so it can be removed so that an IV can be generated.
Template #5 Issue #2
Jun 10, 2017
You don't undertand me previously:
This is not embed on source page. Source page contains <video> tag with src file.
And such content is supported.
Accepted by admin
The issue reporter's template handles this video well:

https://instantview.telegram.org/contest/today.com/template10/?url=http%3A%2F%2Fwww.today.com%2Fstyle%2Fsee-2-women-get-celeb-worthy-ambush-makeovers-t112004
Template #5 Issue #3
Jun 11, 2017
Accepted by admin
This is not really a major issue, because that's the title set in the meta tags. But since the issue was already declined because of another issue, it might be worth fixing it aswell.
Template #9 Issue #1
Jun 13, 2017
Котолий
http://prntscr.com/fjekd9
Accepted by admin
Template #13 handles the case better.
Template #10 Issue #1
Jun 14, 2017
subtitles lost
Accepted by admin
Template #11 Issue #1
Jun 14, 2017
Here is unsupported pinterest embed
Sergey Popkov
unsupported widgets that are not essential for the understanding of the article can be removed
https://instantview.telegram.org/checklist#6-8-3-auxiliary-widgets
Accepted by admin
The content of Pinterest is important to the article. It should not be removed, and anyway there is no way to know if another Pinterest widget may or may not be relevant in another article.
Template #11 Issue #2
Jun 15, 2017
Lol. May be other embeds can be deleted too? Lmao. "Not essential"? This is part of content as video or paragraph
Sergey Popkov
unsupported widgets that are not essential for the understanding of the article can be removed
https://instantview.telegram.org/checklist#6-8-3-auxiliary-widgets
up to admins
Accepted by admin
The content of Pinterest is important to the article. It should not be removed, and anyway there is no way to know if another Pinterest widget may or may not be relevant in another article.
Template #14 Issue #1
Jun 16, 2017
it is cover image
also i think that my template works with recipes better that yours
i lost(dont know how to support video) so i share to you my code for recipes:
?exists: //div[@class="recipe-content-wrapper"]
title: //header[@itemprop="name"]
<figure>: //figure[1]/..
<figcaption>: $$/small
cover: //figure[1][figure]
description: //div[@class="recipe-details-container"]/p[text()]
$prep: //div[@class="recipe-preparation"]
$ingr: //div[@class="recipe-ingredients"]/ul/li

$details: //div[@class="recipe-details-container"]/p[text()]
@prepend_to($ingr): //p[text()="Ingredients"]

@replace_tag(<aside>): //div[@class="recipe-details"]
@replace_tag(<h3>): //p[@class="recipe-title"]
@replace_tag(<li>): //ul
@replace_tag(<p>): //li[@itemprop="recipeIngredient"]
@replace_tag(<h6>): //p[@class="ingredients-title"]
author: //div[@class="recipe-byline"]/div[@itemprop="author"]
@datetime: //time[@class="recipe-time"]
published_date: $@
@remove: //div[@class="recipe-rate"]
@remove: //span[text()="print recipe"]

body: //div
Declined by admin
Cover in this case is optional.

Personally i think it's very nice of you to share your code :) thanks for that nice gesture.
Template #14 Issue #2
Jun 16, 2017
end of code:

body: //div[@class="recipe-details-container"]
@append_to($body): $ingr
@append_to($body): $prep

Please help me. it is for another template. How to make figure with figcaption using image with @alt.
chuchu
Hah. What's the problem?
@wrap(<figure>): $target_img
@after(@alt): $target_img
@wrap(<figcaption>)
and ancestor should not be <p>, make it into div/figure if so
Should work, but depends on where is this image
Declined by admin
Nice that you help each other :) *thumbs up*
Template #14 Issue #3
Jun 17, 2017
Unsupported video on this page. If you decided to delete it as unrelevant to the article than this page also should have IV:
https://instantview.telegram.org/contest/today.com/template14/?url=http%3A%2F%2Fcommunity.today.com%2Fparentingteam%2Fpost%2Fim-a-great-mom-because
Declined by admin
It's a target page and the content you are talking about is a "related video" which is optional and can be removed.

The link you posted is the same as used here?!
Template #14 Issue #4
Jun 17, 2017
Either both of pages (this page and page http://community.today.com/parentingteam/post/im-a-great-mom-because) should have IV or both of them souldn't
chuchu
That's not a big deal. They r using different ancestors for words "Related video" or smthg. Anyway if I catch it then content will be displayed, if not then there won't be IV. All good
Declined by admin
Issues related to missing content might be considered if a link to a template which handles this better is provided.
Template #14 Issue #5
Jun 17, 2017
chuchu
It is optional
Declined by admin
"Related video" content is optional.
Template #18 Issue #1
Jun 19, 2017
Counter issue:
As you can see there is no way to catch all related videos
Declined by admin
"Related video" content is optional.