Skip to content

Add OpenGraph-ready showcase previews and update README showcase section #67

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.

Sign up for GitHub

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

Merged
ProfRandom92 merged 1 commit into from
May 12, 2026
Merged

Add OpenGraph-ready showcase previews and update README showcase section #67

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
18 changes: 12 additions & 6 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -57,13 +57,19 @@ Comptextv7 does **not** claim production Daimler integration, certification, fle

---

## Showcase
## Showcase Preview

Premium static showcase assets are included for GitHub social preview, README review, and OpenGraph-ready demo surfaces.

![Comptextv7 social preview](showcase/assets/social-preview.svg)

| Dashboard preview | Architecture preview |
| --- | --- |
| ![Comptextv7 dashboard preview](showcase/assets/dashboard-preview.svg) | ![Comptextv7 architecture preview](showcase/assets/architecture-preview.svg) |

---

> **Screenshot placeholder**
>
> Add the current Vercel showcase screenshot here when the UI settles:
>
> `docs/assets/showcase-home.png`
## Showcase

@gemini-code-assist gemini-code-assist Bot May 12, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

The heading ## Showcase is redundant given the ## Showcase Preview section just above it. To improve clarity and better describe this section's content (a list of links), consider renaming it.

Suggested alternatives:

  • ## Demo Resources
  • ## Quick Links
Suggested change
## Showcase
## Demo Resources

@gemini-code-assist gemini-code-assist Bot May 12, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

The PR introduces a new ## Showcase Preview section but also adds a ## Showcase header for the links table. This results in two sections with nearly identical names, which can be confusing. Consider renaming this second header to something more descriptive like ## Showcase Resources or ## Reviewer Links to better distinguish it from the visual preview section.

Suggested change
## Showcase
## Showcase Resources


| Reviewer path | Link |
| --- | --- |
Expand Down
36 changes: 36 additions & 0 deletions showcase/assets/architecture-preview.svg
View file
Open in desktop
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
48 changes: 48 additions & 0 deletions showcase/assets/dashboard-preview.svg
View file
Open in desktop
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Loading