Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

Cody pro icon was squashed in the upgrade banner#63356

Merged
rrhyne merged 1 commit into
mainfrom
rrhyne/fix-icon
Jun 19, 2024
Merged

Cody pro icon was squashed in the upgrade banner#63356
rrhyne merged 1 commit into
mainfrom
rrhyne/fix-icon

Conversation

@rrhyne

@rrhyne rrhyne commented Jun 19, 2024

Copy link
Copy Markdown

The Cody PRO icon is now not squashed.
The Cody PRO icon was squashed in the upgrade banner as it was incorrectly specified in Figma.

Test plan

I will manually test to ensure the icon is not squashed in width or height.

@cla-bot cla-bot Bot added the cla-signed label Jun 19, 2024
@rrhyne rrhyne requested review from chrsmith and vdavid June 19, 2024 19:21
@rrhyne

rrhyne commented Jun 19, 2024

Copy link
Copy Markdown
Author

@chrsmith Q: does this NEED a changelog entry? Is the domain correct? There is no listing of domains in the template.

@chrsmith chrsmith left a comment

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.

LGTM. For reference, this is what the "squashed" version looks like.

image

@chrsmith

Copy link
Copy Markdown
Contributor

@chrsmith Q: does this NEED a changelog entry? Is the domain correct? There is no listing of domains in the template.

I would just ignore it. This doesn't seem important enough to warrant calling out in the changelog.

@rrhyne

rrhyne commented Jun 19, 2024

Copy link
Copy Markdown
Author

Gracias!

@rrhyne rrhyne enabled auto-merge (squash) June 19, 2024 19:26
@rrhyne rrhyne merged commit d4948d2 into main Jun 19, 2024
@rrhyne rrhyne deleted the rrhyne/fix-icon branch June 19, 2024 19:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants