URL alias can change, but FB Instant Articles uses canonical URL as key for API interaction. Therefore we need to pass an unchanging canonical URL.

Comments

anbarasan.r created an issue. See original summary.

anbarasan.r’s picture

anbarasan.r commented
Assigned: Unassigned » anbarasan.r
anbarasan.r’s picture

anbarasan.r commented
Assigned: anbarasan.r » Unassigned
Status: Active » Needs review

PR ready to review . https://github.com/BurdaMagazinOrg/module-fb_instant_articles/pull/27

m4olivei’s picture

m4olivei
he/him
Primary language English
Location Grimsby, ON
commented

fb_instant_articles_views might also need to be touched by this as well. In the feed, for each item, you can have an
and element. Do they serve the same purpose? If they do, we should maybe drop use of or maybe make the same as
.

https://github.com/BurdaMagazinOrg/module-fb_instant_articles/blob/7.x-1...

anbarasan.r’s picture

anbarasan.r commented

yes that's correct, we need to change it to the fb_instant_articles_views as well.

scottrigby’s picture

scottrigby
he/him
Primary language English
Location Brooklyn, NY
commented
Issue tags: +7.x-1.0-rc1 blocker
anbarasan.r’s picture

anbarasan.r commented
Status: Needs review » Closed (works as designed)

Closing this issue based on the GH discussion, refer : https://github.com/BurdaMagazinOrg/module-fb_instant_articles/pull/27