- Issue created by @alemadlei
- last update
11 months ago 61 pass, 2 fail - Status changed to Needs review
10 months ago 7:41pm 24 January 2024 - last update
9 months ago 61 pass, 2 fail - Issue was unassigned.
- ๐บ๐ธUnited States DamienMcKenna NH, USA
Thank you for the merge request, we'll try to review it soon.
As a reminder, when you're finished with your work please set the "Assigned" field to "Unassigned".
- last update
5 months ago 61 pass, 2 fail - ๐ณ๐ฑNetherlands Drumanuel
Got warnings for offers and hasCourseInstance
Here an issue has been logged for hasCourseInstance
https://www.drupal.org/project/schema_metatag/issues/3410151 โจ Missing field 'hasCourseInstance' Active - Status changed to Needs work
4 months ago 4:08pm 20 July 2024 - ๐บ๐ธUnited States DamienMcKenna NH, USA
Looks pretty good, but it needs a test file to make sure the functionality works.
- ๐ซ๐ทFrance Arono974
Hello,
I have the same problem with :
Missing "offers" fields :
Missing "HasCourseInstance" fields :
Have you found a way to fix this? - ๐ณ๐ฑNetherlands bram.velthoven
bram.velthoven โ made their first commit to this issueโs fork.
- ๐ณ๐ฑNetherlands bram.velthoven
@Arono_ I've comitted to the MR and added the course tags HasCourseInstance and Offers
- ๐ฌ๐งUnited Kingdom the_g_bomb
I have had a quick review and suspect that while this may quieten google from complaining, it not be that useful.
According to: https://schema.org/hasCourseInstance
"hasCourseInstance" should be of "@type": "CourseInstance". Also I suspect there should be a way to set some other variables here as well.The courseInstance should also be repeatable according to:
https://developers.google.com/search/docs/appearance/structured-data/cou...This suggests at the very least the definition should look a bit more like:
* multiple = TRUE, * property_type = "type", * tree_parent = { * "CourseInstance", * },
- ๐ฌ๐งUnited Kingdom the_g_bomb
I have added the new items to the config/schema file as I think they may have been missed.
Tweaked the annotations to fix some typos left over from a copy process.
Added a new PropertyType for courseInstance and also updated the SchemaCourseOffers to allow multiples of the offer property.
The new course instance has the bits I need for now, but could be updated easily to add more as defined by: