Skip to content

Add new icons for Segmentation templates #2779

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.

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
merged 1 commit into from
Apr 23, 2025

Conversation

aishad
Copy link
Contributor

@aishad aishad commented Apr 16, 2025

Partially fixes:https://github.com/Shopify/core-issues/issues/87053

Add icons for new templates:

Icon Template
Language image
PersonAdd image
Apps image

🎩

  • ...

Checklist

  • I have 🎩'd these changes
  • I have updated relevant documentation

@aishad aishad force-pushed the ad/segmentation-templates-new-icons branch from 720c04f to 5929f31 Compare April 16, 2025 16:38
@aishad aishad marked this pull request as ready for review April 16, 2025 16:42
'@shopify/ui-extensions': minor
---

Passing new icon types to the segment templates
Copy link
Contributor

Choose a reason for hiding this comment

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

small nit:

Suggested change
Passing new icon types to the segment templates
Add new icon types for segment templates

Copy link
Contributor

@PrescilliaA PrescilliaA left a comment

Choose a reason for hiding this comment

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

LGTM!

@aishad aishad force-pushed the ad/segmentation-templates-new-icons branch from 5929f31 to cb0f787 Compare April 17, 2025 20:14
@aishad aishad force-pushed the ad/segmentation-templates-new-icons branch from cb0f787 to 3d8a237 Compare April 23, 2025 18:25
@aishad aishad requested a review from LTiger14 April 23, 2025 18:26
@aishad aishad force-pushed the ad/segmentation-templates-new-icons branch from 3d8a237 to aa563e1 Compare April 23, 2025 18:51
Copy link
Contributor

@LTiger14 LTiger14 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, just make sure to change products to product

@aishad aishad force-pushed the ad/segmentation-templates-new-icons branch from aa563e1 to 742eb03 Compare April 23, 2025 20:17
@aishad aishad merged commit c410a02 into unstable Apr 23, 2025
6 checks passed
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.

3 participants