Skip to content

fix(sideNav): revert temp fix for tokens update#4270

Merged
evwilkin merged 7 commits intopatternfly:mainfrom
tlabaj:sidenav_tokens
Sep 23, 2024
Merged

fix(sideNav): revert temp fix for tokens update#4270
evwilkin merged 7 commits intopatternfly:mainfrom
tlabaj:sidenav_tokens

Conversation

@tlabaj
Copy link
Contributor

@tlabaj tlabaj commented Sep 19, 2024

closes #4264

@tlabaj tlabaj requested review from dlabaj and evwilkin September 19, 2024 13:55
@evwilkin
Copy link
Member

@tlabaj this was failing CI as the versions weren't bumped to pull in the fix from react-tokens. I pulled it locally and got it working after a few tries so pushed the version bumps back up to this PR to see if we can get CI passing.

@evwilkin
Copy link
Member

@tlabaj this CI error is what we see what the @patternfly/documenation-framework version isn't matched in documentation-site's package.json dependency - can you bump that to alpha 97 so they match?

@patternfly-build
Copy link
Collaborator

Copy link
Member

@evwilkin evwilkin left a comment

Choose a reason for hiding this comment

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

LGTM! 🎉

@evwilkin evwilkin merged commit 4ccd2f0 into patternfly:main Sep 23, 2024
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.

side nav: update to use prefixed token

3 participants