Skip to content

Updating screenshots for Data Products to Tracking Plans#1613

Merged
jborlase-snowplow merged 2 commits intomainfrom
update-data-product-screenshots
Feb 3, 2026
Merged

Updating screenshots for Data Products to Tracking Plans#1613
jborlase-snowplow merged 2 commits intomainfrom
update-data-product-screenshots

Conversation

@jborlase-snowplow
Copy link
Contributor

What changed?

Why?

Reviewer guidance

AI reviews

Claude will automatically review this PR against the docs style guide.

If you have questions or want it to look again at something specific, tag @claude in a comment.

@netlify
Copy link

netlify bot commented Feb 2, 2026

Deploy Preview for snowplow-docs ready!

Name Link
🔨 Latest commit c4b04d4
🔍 Latest deploy log https://app.netlify.com/projects/snowplow-docs/deploys/6980c0bb11d4f300089607ad
😎 Deploy Preview https://deploy-preview-1613--snowplow-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 20 (no change from production)
Accessibility: 91 (no change from production)
Best Practices: 92 (no change from production)
SEO: 95 (no change from production)
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

@claude
Copy link

claude bot commented Feb 2, 2026

Style Guide Review

I've reviewed the changes in this PR against the Snowplow documentation style guide. Here are some suggestions for improvement:

Writing style issues

Line 48: Remove filler word

Current: "If you need to edit a tracking plan at any time, simply select it..."
Suggested: "If you need to edit a tracking plan at any time, select it..."

Remove "simply" - it's a filler word that should be avoided per the style guide.

Line 88: Typo

Current: "The conflict resolution UI provides options to the user tp modify..."
Suggested: "The conflict resolution UI provides options to the user to modify..."

Fix typo: "tp" → "to"

Formatting issues

Line 21: Unnecessary italics

Current: _The name of your tracking plan must be unique..._
Suggested: The name of your tracking plan must be unique...

The note admonition already provides visual distinction. The italic formatting is unnecessary and not standard for note content.

Line 42: Asterisk for note

Current: *Notes: sharing and subscribing is only available...*
Suggested: :::note
Sharing and subscribing is only available for users registered in Snowplow Console.
:::

Use proper admonition syntax instead of asterisk-based italic text. Also, remove the label "Notes:" (admonitions speak for themselves), and fix subject-verb agreement: "sharing and subscribing" is plural, so use "are" not "is".

Line 66: Missing article

Current: "...along with a **'View Changes'**."
Suggested: "...along with a **View Changes** button."

Complete the sentence with "button" and remove the quotes around the UI element name (bold is sufficient for UI elements).

Line 88: Grammar issue

Current: "...options to the user tp modify or delete each instruction..."
Suggested: "...options to modify or delete each instruction..."

"To the user" is redundant - the options are clearly for the user. Simplify to "options to modify..."

Copy link
Contributor

@gibbok-snowplow gibbok-snowplow left a comment

Choose a reason for hiding this comment

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

LGTM thanks!

@jborlase-snowplow jborlase-snowplow merged commit 535979a into main Feb 3, 2026
6 checks passed
@jborlase-snowplow jborlase-snowplow deleted the update-data-product-screenshots branch February 3, 2026 09:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants