-
Notifications
You must be signed in to change notification settings - Fork 67
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Replace meta.html
with a more maintenable / extendable approach
#94
Comments
Same issue here. I'm just adding unsupported meta tags manually in the template now cause django-meta doesnt support the Facebook/OpenGraph metatags/microtags for product catalog registration. Would be great if one could just add new tags via the MetaMixin. |
@lggwettmann thanks for the feedback. You can also add custom tags via |
Tried that, but didn't show up in the meta tags. |
Would it perhaps be an idea to start implementing the JSON-LD instead or as an addition to the basic meta tags? I could imagine that that would bring quite an advantage nowadays and should be (?) more extendable? I haven't done much research and just stumbled upon JSON-LD though. |
JSON-LD support is in the works in #76 |
@lggwettmann are you using views or models? |
@lggwettmann you may want to check #95 (and https://github.com/nephila/django-meta/pull/95/files#diff-823680934ff274e0f1ecdacdf80a9f61R3 in detail) regarding implementing extra / custom tags properties |
meta.html
is quite burdensome to support, as any change to will require people who extended it to adapt.We will likely need a more robust solution that will allow people to extend it, without the need to change it everytime we add more tags
The text was updated successfully, but these errors were encountered: