Smalyshev added a comment.
In general, yes, but see my comments on the patch. PropertyRdfBuilder::propertyIsLink should be updated. If we don't have any GeoShape properties yet, it doesn't matter too much, and yes, we'll reload for T166244 as soon as we merge the patch.TASK
daniel edited subtasks, added: T163921: Implement canonical data URLs for machine readable page content; removed: T159517: [RFC] RDF mapping for geo-shape / URIs for commons data pages.
TASK DETAILhttps://phabricator.wikimedia.org/T160535EMAIL
gerritbot added a project: Patch-For-Review.
TASK DETAILhttps://phabricator.wikimedia.org/T160535EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: gerritbotCc: gerritbot, Anchal09, daniel, Aklapper, Lydia_Pintscher, Adik2382, Th3d3v1ls, Ramalepe, Liugev6,
daniel added a subtask: T159517: [RFC] RDF mapping for geo-shape.
TASK DETAILhttps://phabricator.wikimedia.org/T160535EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: danielCc: Anchal09, daniel, Aklapper, Lydia_Pintscher, QZanden, D3r1ck01, Izno,