Skip to content
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

Jump to bottom

Bump noredink-ui to 29.1.0 to include new StickerIcons and UiIcons for annotations #1686

Merged
merged 2 commits into from
Jun 28, 2024
Merged

Bump noredink-ui to 29.1.0 to include new StickerIcons and UiIcons for annotations #1686

merged 2 commits into from
Jun 28, 2024

Conversation

micahhahn
Copy link
Member

@micahhahn micahhahn commented Jun 28, 2024 •
edited
Loading

🏷️ Bump for version 29.1.0

What changes does this release include?

How has the API changed?

---- Nri.Ui.UiIcon.V1 - MINOR ----

    Added:
        addSticker : Nri.Ui.Svg.V1.Svg
        circle : Nri.Ui.Svg.V1.Svg
        delete : Nri.Ui.Svg.V1.Svg

Releasing

After this PR merges, and you've pulled down latest master, finish following the publishing process.

@linear Linear
Copy link

linear bot commented Jun 28, 2024

PHX-1669 Add sticker SVG assets to noredink-ui

@micahhahn micahhahn changed the title Bump noredink-ui to 29.1.0 Bump noredink-ui to 29.1.0 to include new StickerIcons and UiIcons for annotations Jun 28, 2024
@erik-nri erik-nri self-requested a review June 28, 2024 22:14
erik-nri
erik-nri approved these changes Jun 28, 2024
View reviewed changes
Copy link
Contributor

@erik-nri erik-nri left a comment

Choose a reason for hiding this comment

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

:shipit:

@micahhahn micahhahn marked this pull request as ready for review June 28, 2024 22:17
@micahhahn micahhahn merged commit bfd714f into master Jun 28, 2024
5 of 7 checks passed
@micahhahn micahhahn deleted the phx-1669-bump-noredink-ui branch June 28, 2024 22:17
@micahhahn micahhahn mentioned this pull request Jul 3, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@juanedi juanedi juanedi approved these changes

@erik-nri erik-nri erik-nri approved these changes

@staceyadams staceyadams Awaiting requested review from staceyadams

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@micahhahn @juanedi @erik-nri