Skip to content

fix: indexer diagram#395

Merged
jeff-matthews merged 3 commits intomainfrom
GH-394-indexer-diagram
Jun 19, 2025
Merged

fix: indexer diagram#395
jeff-matthews merged 3 commits intomainfrom
GH-394-indexer-diagram

Conversation

@jeff-matthews
Copy link
Copy Markdown
Contributor

Purpose of this pull request

This pull request (PR) fixes #394.

I've tried using the SVG, but Gatsby seems to do something weird with the format and it doesn't render properly. I exported the new PNG from the SVG with the updated content. This should be sufficient for now. At least we'll have the SVG to make changes in the future if we need to.

@jeff-matthews jeff-matthews self-assigned this Jun 19, 2025
@jeff-matthews jeff-matthews added content-quality Language, formatting, SEO metadata, authoring standards compliance internal Differentiates work between external and internal contributors labels Jun 19, 2025
@jeff-matthews jeff-matthews requested a review from jhadobe June 19, 2025 15:03
Copy link
Copy Markdown
Contributor

@jhadobe jhadobe left a comment

Choose a reason for hiding this comment

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

@jeff-matthews you might want to resize, or switch to horizontal. it looks a little weird on the page:

indexer-image

@github-project-automation github-project-automation bot moved this from 📋 Needs Review to 👍 Approved in Commerce - Pull Requests Jun 19, 2025
@jeff-matthews jeff-matthews merged commit 5e77c04 into main Jun 19, 2025
5 checks passed
@github-project-automation github-project-automation bot moved this from 👍 Approved to 🏁 Done in Commerce - Pull Requests Jun 19, 2025
@jeff-matthews jeff-matthews deleted the GH-394-indexer-diagram branch June 19, 2025 15:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

content-quality Language, formatting, SEO metadata, authoring standards compliance internal Differentiates work between external and internal contributors

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

Diagram has wrong information about marking indexer as invalid.

2 participants