The Twitter Cards specifications indicate that "twitter:image:src" should be used, so "twitter:image" seems to have been dropped or was a mistake.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

DamienMcKenna’s picture

DamienMcKenna’s picture

Status: Active » Needs review
FileSize
2.36 KB

This patch replaces the meta tag, updates the README.txt, and adds an update script to replace update existing records.

Status: Needs review » Needs work

The last submitted patch, 2: metatag-n2267501-2.patch, failed testing.

DamienMcKenna’s picture

DamienMcKenna’s picture

Status: Needs work » Postponed
DamienMcKenna’s picture

FileSize
4.63 KB

This improves the update script so that it also fixes metatag_config, and then also fixes the default value from metatag_twitter_cards_metatag_bundled_config_alter.

DamienMcKenna’s picture

Status: Postponed » Needs review

Unblocked.

DamienMcKenna’s picture

Status: Needs review » Needs work
DamienMcKenna’s picture

Status: Needs work » Needs review

  • DamienMcKenna committed 7fab856 on 7.x-1.x
    Issue #2267501 by DamienMcKenna: Renamed the 'twitter:image' meta tag to...
DamienMcKenna’s picture

Status: Needs review » Fixed

Committed, though I did split the update script in two, just to avoid possible timeouts.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

Andrew_Mallis’s picture

Priority: Major » Minor
Status: Closed (fixed) » Needs work

Looks like Twitter spec is now twitter:image again instead of twitter:image:src.
Not seeing anything breaking currently.

DamienMcKenna’s picture

Priority: Minor » Major
Status: Needs work » Closed (fixed)