Skip to content

Add home page in academy-theme#184

Open
CodexRaunak wants to merge 2 commits intolayer5io:masterfrom
CodexRaunak:add-home-page
Open

Add home page in academy-theme#184
CodexRaunak wants to merge 2 commits intolayer5io:masterfrom
CodexRaunak:add-home-page

Conversation

@CodexRaunak
Copy link
Copy Markdown
Contributor

Notes for Reviewers

Fixes #183

It moves the default Academy homepage content into academy-theme so Academy repositories can inherit a shared homepage instead of duplicating the same copy in each repo.

Changes include:

  • Adds a theme-level content/_index.md with shared Academy homepage content.
  • Adds a theme-level layouts/index.html that renders the homepage content with {{ .Content }}.
  • Add Best Practices for authors to validate and create IDs from the Academy Instructor Console.

Signed commits

  • Yes, I signed my commits.

Signed-off-by: Raunak Madan <madanraunak24@gmail.com>
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the .gitignore to track the content/_index.md file while ignoring other content, adds a new development landing page for the Academy, and provides a basic Hugo layout for the homepage. The review feedback suggests fixing typos and clarifying the purpose of the index page, correcting the heading hierarchy for better accessibility, and replacing a deep-link Slack redirect with a more reliable public invite link.

Comment thread content/_index.md Outdated
Comment thread content/_index.md Outdated
Comment thread content/_index.md Outdated
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Signed-off-by: Raunak Madan <143412156+CodexRaunak@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Docs] Centralize Academy homepage content in academy-theme

1 participant