Skip to content

Comments

ENG-6355: template for highlight pages#1440

Merged
LineIndent merged 6 commits intomainfrom
ahmad/make-page-template
Jun 21, 2025
Merged

ENG-6355: template for highlight pages#1440
LineIndent merged 6 commits intomainfrom
ahmad/make-page-template

Conversation

@LineIndent
Copy link
Contributor

No description provided.

@linear
Copy link

linear bot commented Jun 19, 2025

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

PR Summary

Major architectural change introducing a new highlight page template system and reorganizing use cases documentation for better maintainability and user experience.

  • Added pcweb/templates/highlightpage.py introducing a flexible decorator-based template system for consistent page layouts
  • Relocated and expanded use cases content from docs to pcweb/pages/use_cases/use_cases.md with comprehensive industry examples
  • Refactored Databricks page in pcweb/pages/databricks/databricks.py to use new template system
  • Updated navigation links in footers to reflect new /use-cases route structure
  • Implemented dark mode toggle in customer views footer at pcweb/pages/customers/views/footer.py

8 files reviewed, 1 comment
Edit PR Review Bot Settings | Greptile

@carlosabadia
Copy link
Collaborator

@LineIndent Lets remove the ai-builder use-cases page and sidebar item

Copy link
Collaborator

@carlosabadia carlosabadia left a comment

Choose a reason for hiding this comment

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

Looks good, we can merge the new landing page first tho

@LineIndent LineIndent merged commit a604517 into main Jun 21, 2025
9 checks passed
@LineIndent LineIndent deleted the ahmad/make-page-template branch June 21, 2025 06:50
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