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

Fix missing _internal/google_analytics_async.html reference #70

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

divine7022
Copy link

🔹 PR Description:

While setting up the project locally, I encountered an error related to the missing _internal/google_analytics_async.html template in the Compose theme.

🔹 Error Message:

error calling partial: execute of template failed: html/template:partials/opengraph.html:5:13: 
no such template "_internal/google_analytics_async.html"

🔹 Possible Cause:

The Hugo _internal/google_analytics_async.html partial was deprecated and removed in newer versions of Hugo. However, the opengraph.html file in the theme still references it, causing the build to fail.

🔹 Fix Implemented:

  • Removed the reference to _internal/google_analytics_async.html in themes/compose/layouts/partials/opengraph.html .

Screenshot 2025-02-15 004005

  • Ensured compatibility with the latest Hugo versions to prevent build failures.

🔹 Verification Steps:

  • Ran hugo serve to confirm the issue was resolved.
  • Checked that the website functions correctly without the missing reference.

Let me know if any further modifications are required. 🚀

@divine7022
Copy link
Author

@Akshatji800 Looking forward to your response on this.
Thank you in advance

@cfsmp3 cfsmp3 requested a review from techno-disaster March 1, 2025 06:29
@techno-disaster
Copy link
Member

@divine7022 could you quickly rebase please?

@divine7022
Copy link
Author

@divine7022 could you quickly rebase please?

@techno-disaster i resolved the merge conflict so could you please review once again and let me know any further modifications are needed.

@techno-disaster
Copy link
Member

Again, a merge not a rebase

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