Updated: Comment #N

Problem/Motivation

Support the new 'See It' Twitter Card being introduced by the Comcast/Twitter agreement.

The new card type is called 'seeit'.

The new properties are:

  • showid*
  • idspace*
  • campaignid
  • assetid

* required

Twitter: https://blog.twitter.com/2013/change-the-channel-directly-from-twitter
Comcast: http://corporate.comcast.com/seeit

I created a submodule of the Twitter Cards module vs integrating it into the Twitter Cards module directly because See It may not be used by the majority of websites (outside the TV industry) and I don't want to force it on anyone who just wants a normal Twitter card.

The See It button is aiming to become an industry standard and not a proprietary exclusive technology so this will eventually benefit more than just Comcast/NBCUniversal. However, I can see the need to hold off merging this in until more happens after the initial roll out. Until then at least this will be here as a patch for others to use if they need to implement this into their Drupal site.

Proposed resolution

Create submodule. Patch below.

Remaining tasks

none

User interface changes

none

API changes

none

none

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Elijah Lynn’s picture

Here is a patch for the request above.

Elijah Lynn’s picture

Issue summary: View changes
Status: Active » Needs review
Related issues: +#1963678: Add support for new Twitter Cards types

Setting to needs review since we have another patch for the new card types also posted. This is one of the new card types that just isn't yet published (I think).

Elijah Lynn’s picture

Elijah Lynn’s picture

Status: Active » Needs review
DamienMcKenna’s picture

Given how limited the use case is, this should be a separate module, at least for now.

DamienMcKenna’s picture

Status: Needs review » Postponed