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

Update docs.md #1113

Merged
merged 1 commit into from
Jan 6, 2025
Merged

Update docs.md #1113

merged 1 commit into from
Jan 6, 2025

Conversation

ganakee
Copy link
Contributor

@ganakee ganakee commented Jan 4, 2025

In the manual (non-CLI) template customization instructions, the original instructions result in a fatal error after setting-up the inherited theme. The error is: Template "@images/grav-logo.svg" is not defined in "partials/logo.html.twig" at line 7.

I traced the error to a typo or change in the original instructions on this page. The original instructions at the prefixes section omitted using a single-quote around the prefixes and the path to the themes. Thus, on activating the theme, GRAV found nothing to inherit and failed with a fatal error. The correction is minor but critical.

In the manual (non-CLI) template customization instructions, the original instructions result in a fatal error after setting-up the inherited theme. The error is: Template "@images/grav-logo.svg" is not defined in "partials/logo.html.twig" at line 7.

I traced the error to a typo or change in the original instructions on this page.  The original instructions at the prefixes section omitted using a single-quote around the prefixes and the path to the themes. Thus, on activating the theme, GRAV found nothing to inherit and failed with a fatal error. The correction is minor but critical.
@rhukster rhukster merged commit 351cb9e into getgrav:develop Jan 6, 2025
@rhukster
Copy link
Member

rhukster commented Jan 6, 2025

thanks.

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.

2 participants