Skip to content
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

Unify the $page property and add a fluent metadata helper #388

Merged
merged 22 commits into from
May 17, 2022

Conversation

caendesilva
Copy link
Member

Breaking changes

The $post property has been renamed to $page to unify Blade layouts. The actual contents of the property is the same, however any published views must be changed. Since the meta.blade.php view and the config/hyde.php have both been changed they too need to be republished.

Metadata helper

Intended to be used in configs, the Meta helper gives an easy programmatic way to render HTML metadata.

Plan is to merge this with blog post data. As it is now it does not take care of removing duplicates in blog posts.

Usage:

Meta::name('foo', 'bar'),
Meta::property('foo', 'bar'),

Generates:

<meta name="foo" content="bar">
<meta property="og:foo" content="bar">

These are then now used in the Hyde configuration.

Fixes

Fix #382
Fix #375

@caendesilva caendesilva marked this pull request as ready for review May 17, 2022 20:44
@caendesilva caendesilva merged commit 6aa83b6 into master May 17, 2022
@caendesilva caendesilva deleted the fluent-metadata-handler branch May 17, 2022 20:47
caendesilva pushed a commit that referenced this pull request Aug 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unify the $page property Add config option to add og:properties, like we have for meta tags
2 participants