Skip to content

Conversation

kimberleyamackenzie
Copy link
Contributor

Proposed changes

Unreleased product version (optional)

Related issues (optional)

Copy link
Contributor

claude bot commented Sep 26, 2025

PR Review Summary

This PR updates promotional content across the site to advertise the new AI Agentic Workflows course. The changes look good overall but I found a few issues to address:

Issues Found:

File ending violations (critical):

  • layouts/partials/docs/right-nav-ad.html:5 - Missing newline at end of file
  • layouts/partials/tutorials/right-nav-ad.html:5 - Missing newline at end of file

Content/Style:

  • layouts/partials/header.html:180 - Extra leading space in subtitle: ' Zero to Production with AI Agentic Workflows' should be 'Zero to Production with AI Agentic Workflows'

Tracking attribute inconsistency:

  • Both ad files still use data-track="docs-neo-ad" but are now promoting the AI course, not Neo. Consider updating to something like data-track="docs-ai-course-ad" for better analytics tracking.

Positive aspects:

  • All image references are valid and the new image file exists
  • HTML structure and formatting are consistent
  • Alt text is descriptive and appropriate
  • URL structure follows expected patterns
  • Banner text is clear and action-oriented

Recommendations:

  1. Add missing newlines to the two ad template files
  2. Remove the extra space in the course subtitle
  3. Update the tracking attribute for consistency
  4. Consider testing the course URL accessibility

@pulumi-bot
Copy link
Collaborator

@kimberleyamackenzie
Copy link
Contributor Author

@SaraDPH I'm looking into adding the ad to Registry too, but that happens in a different repo and it appears there is a bug there preventing the ads from being shown - I'm diving deeper, but I may not be able to get that done for Monday.

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