Visitar URL original
3.1.0 release updates (second attempt) by thisisdano · Pull Request #4891 · uswds/uswds · GitHub
Skip to content

3.1.0 release updates (second attempt) - #4891

Merged
thisisdano merged 4 commits into
developfrom
dw-3.1.0-release-updates-2
Aug 1, 2022
Merged

thisisdano merged 4 commits into
developfrom
dw-3.1.0-release-updates-2

Conversation

@thisisdano

@thisisdano thisisdano commented Jul 29, 2022 •

Copy link
Copy Markdown
Contributor

This is a second attempt to add release updates for 3.1.0.

NOTE: In my first attempt, there was an emoji reference I did not remember adding. This was potential user error, but I'm creating a new PR just in case.

  • Update references from 3.0.2 → 3.1.0
  • Add notifications for potential breaking changes

@thisisdano
thisisdano requested review from amyleadem and mejiaj July 29, 2022 18:39
+ "\A - Settings: We changed the default value of $theme-hero-image."
+ "\A Teams that use the default image (hero.png) will need to get the"
+ "\A new image (hero.jpg) from the distribution package."
+ "\A ";

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.

We moved @mixin embed-container to deprecated.scss in the aspect-ratio PR. Not sure that it would warrant a notification, but wanted to share just in case.

https://github.com/uswds/uswds/pull/4811/files#diff-c2c5b5639aae3863e94643a3a613d89272dd926b077aae1c743a0eecabb9d355

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I've pushed a change adding this deprecation notice

@amyleadem amyleadem 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.

Looks good to me! One small issue with a possible extra space, but the notification content seems to contain everything it needs.

Co-authored-by: Amy Leadem <93996430+amyleadem@users.noreply.github.com>
@thisisdano
thisisdano merged commit f2752b2 into develop Aug 1, 2022
@thisisdano
thisisdano deleted the dw-3.1.0-release-updates-2 branch August 1, 2022 17:08
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