-
Notifications
You must be signed in to change notification settings - Fork 60
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
chore(deps): update theming-base-content 11.20.0 [ci visual] #5748
Conversation
✅ Deploy Preview for fundamental-styles ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
packages/styles/stories/Components/Icons/icon/BusinessSuiteInAppSymbols/dataBusinessSuite.json
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
see my comments
non-stackable: "\e185" | ||
non-stackable: "\e185", | ||
back-to-search: "\e186", | ||
business-objects-folder: "\e187" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please add a comma at the end, so next update doesn't detect this as a new change
"cross-target" | ||
"cross-target", | ||
"back-to-search", | ||
"business-objects-folder" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please add a comma at the end
8c6dbb2
to
7297902
Compare
7297902
to
1388a1f
Compare
update theming-base-content 11.20.0