microlinkhq / metascraper

Get unified metadata from websites using Open Graph, Microdata, RDFa, Twitter Cards, JSON-LD, HTML, and more.
https://metascraper.js.org
MIT License
2.35k stars 168 forks source link

fix: use name instead of property #383

Closed Kikobeats closed 3 years ago

Kikobeats commented 3 years ago

This is a continuation of https://github.com/microlinkhq/metascraper/pull/379 but for metascraper-audio

Kikobeats commented 3 years ago

waiting until @goleary's opinion 🙂

goleary commented 3 years ago

As I mention here, I don't think we should be modifying the rule to use name instead of property.

We should be adding a rule that allows using name in addition to property. This will prevent sites that are working today from being broken.