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.

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

Bump the patternfly group in /ui with 7 updates #994

Merged
merged 4 commits into from
Sep 3, 2024

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Aug 28, 2024

Bumps the patternfly group in /ui with 7 updates:

Package From To
@patternfly/patternfly 5.3.1 5.4.0
@patternfly/react-charts 7.3.0 7.4.0
@patternfly/react-core 5.3.4 5.4.0
@patternfly/react-drag-drop 5.3.4 5.4.0
@patternfly/react-icons 5.3.2 5.4.0
@patternfly/react-table 5.3.4 5.4.0
@patternfly/react-tokens 5.3.1 5.4.0

Updates @patternfly/patternfly from 5.3.1 to 5.4.0

Release notes

Sourced from @​patternfly/patternfly's releases.

v5.4.0

What's Changed

... (truncated)

Commits
  • f75001a Update issue templates
  • a9e2a7a Update issue templates
  • c0c2559 Update and rename PF6_alpha_bug_request.md to PF6_beta_bug_request.md (#6935)
  • fec6266 chore(deps): update dependency @​patternfly/documentation-framework to v5.16.1...
  • 1550c08 chore(deps): update dependency @​patternfly/documentation-framework to v5.16.1...
  • ca9b9e6 fix(InlineEdit): updated example to match React (#6877)
  • d5ee3d9 chore(build): corepack use [email protected] (#6872)
  • 50182be chore(deps): update dependency @​patternfly/react-code-editor to v5.3.4 (#6859)
  • aad6f30 fix(form): support label addl info in horizontal form (#6853)
  • baa5f63 chore(deps): update dependency @​patternfly/react-table to v5.3.4 (#6861)
  • Additional commits viewable in compare view

Updates @patternfly/react-charts from 7.3.0 to 7.4.0

Commits
  • 3ef073a chore(release): releasing packages [ci skip]
  • 7a25feb chore: Update promote.sh to promote to npm.
  • 494143d chore: Update promote.sh with dry run
  • 5836008 Revert "chore(release): Update promote.sh"
  • 5937183 chore(release): Update promote.sh
  • 701ccd9 Revert "chore(release): releasing packages [ci skip]"
  • 48c08cb chore(release): releasing packages [ci skip]
  • a76ba5e chore(release): releasing packages [ci skip]
  • 279c54f fix(Table): remove blank scope in empty deprecated table headers (#10875)
  • 7b64b05 Update issue templates
  • Additional commits viewable in compare view

Updates @patternfly/react-core from 5.3.4 to 5.4.0

Commits
  • 3ef073a chore(release): releasing packages [ci skip]
  • 7a25feb chore: Update promote.sh to promote to npm.
  • 494143d chore: Update promote.sh with dry run
  • 5836008 Revert "chore(release): Update promote.sh"
  • 5937183 chore(release): Update promote.sh
  • 701ccd9 Revert "chore(release): releasing packages [ci skip]"
  • 48c08cb chore(release): releasing packages [ci skip]
  • a76ba5e chore(release): releasing packages [ci skip]
  • 279c54f fix(Table): remove blank scope in empty deprecated table headers (#10875)
  • 7b64b05 Update issue templates
  • Additional commits viewable in compare view

Updates @patternfly/react-drag-drop from 5.3.4 to 5.4.0

Release notes

Sourced from @​patternfly/react-drag-drop's releases.

v5.4.0

What's Changed

... (truncated)

Commits
  • 3ef073a chore(release): releasing packages [ci skip]
  • 7a25feb chore: Update promote.sh to promote to npm.
  • 494143d chore: Update promote.sh with dry run
  • 5836008 Revert "chore(release): Update promote.sh"
  • 5937183 chore(release): Update promote.sh
  • 701ccd9 Revert "chore(release): releasing packages [ci skip]"
  • 48c08cb chore(release): releasing packages [ci skip]
  • a76ba5e chore(release): releasing packages [ci skip]
  • 279c54f fix(Table): remove blank scope in empty deprecated table headers (#10875)
  • 7b64b05 Update issue templates
  • Additional commits viewable in compare view

Updates @patternfly/react-icons from 5.3.2 to 5.4.0

Commits
  • 3ef073a chore(release): releasing packages [ci skip]
  • 7a25feb chore: Update promote.sh to promote to npm.
  • 494143d chore: Update promote.sh with dry run
  • 5836008 Revert "chore(release): Update promote.sh"
  • 5937183 chore(release): Update promote.sh
  • 701ccd9 Revert "chore(release): releasing packages [ci skip]"
  • 48c08cb chore(release): releasing packages [ci skip]
  • a76ba5e chore(release): releasing packages [ci skip]
  • 279c54f fix(Table): remove blank scope in empty deprecated table headers (#10875)
  • 7b64b05 Update issue templates
  • Additional commits viewable in compare view

Updates @patternfly/react-table from 5.3.4 to 5.4.0

Commits
  • 3ef073a chore(release): releasing packages [ci skip]
  • 7a25feb chore: Update promote.sh to promote to npm.
  • 494143d chore: Update promote.sh with dry run
  • 5836008 Revert "chore(release): Update promote.sh"
  • 5937183 chore(release): Update promote.sh
  • 701ccd9 Revert "chore(release): releasing packages [ci skip]"
  • 48c08cb chore(release): releasing packages [ci skip]
  • a76ba5e chore(release): releasing packages [ci skip]
  • 279c54f fix(Table): remove blank scope in empty deprecated table headers (#10875)
  • 7b64b05 Update issue templates
  • Additional commits viewable in compare view

Updates @patternfly/react-tokens from 5.3.1 to 5.4.0

Commits
  • 3ef073a chore(release): releasing packages [ci skip]
  • 7a25feb chore: Update promote.sh to promote to npm.
  • 494143d chore: Update promote.sh with dry run
  • 5836008 Revert "chore(release): Update promote.sh"
  • 5937183 chore(release): Update promote.sh
  • 701ccd9 Revert "chore(release): releasing packages [ci skip]"
  • 48c08cb chore(release): releasing packages [ci skip]
  • a76ba5e chore(release): releasing packages [ci skip]
  • 279c54f fix(Table): remove blank scope in empty deprecated table headers (#10875)
  • 7b64b05 Update issue templates
  • Additional commits viewable in compare view

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore <dependency name> major version will close this group update PR and stop Dependabot creating any more for the specific dependency's major version (unless you unignore this specific dependency's major version or upgrade to it yourself)
  • @dependabot ignore <dependency name> minor version will close this group update PR and stop Dependabot creating any more for the specific dependency's minor version (unless you unignore this specific dependency's minor version or upgrade to it yourself)
  • @dependabot ignore <dependency name> will close this group update PR and stop Dependabot creating any more for the specific dependency (unless you unignore this specific dependency or upgrade to it yourself)
  • @dependabot unignore <dependency name> will remove all of the ignore conditions of the specified dependency
  • @dependabot unignore <dependency name> <ignore condition> will remove the ignore condition of the specified dependency and ignore conditions

@dependabot dependabot bot added dependencies Pull requests that update a dependency file javascript Pull requests that update Javascript code labels Aug 28, 2024
@github-actions github-actions bot added this to the 0.3.2 milestone Aug 28, 2024
@dependabot dependabot bot force-pushed the dependabot/npm_and_yarn/ui/patternfly-be2d9dd4ed branch 3 times, most recently from 5b688ee to 3b62596 Compare August 28, 2024 09:04
@MikeEdgar
Copy link
Member

@dependabot rebase

@dependabot dependabot bot force-pushed the dependabot/npm_and_yarn/ui/patternfly-be2d9dd4ed branch from 3b62596 to fe8969d Compare August 28, 2024 12:27
@MikeEdgar MikeEdgar removed this from the 0.3.2 milestone Aug 28, 2024
@MikeEdgar MikeEdgar force-pushed the dependabot/npm_and_yarn/ui/patternfly-be2d9dd4ed branch from fe8969d to ef7f4cf Compare August 28, 2024 19:13
@MikeEdgar
Copy link
Member

@riccardo-forina apparently a <div id="root">...</div> is required for React apps?

See patternfly/patternfly-react#10283 for the patternfly change that seems to cause the error.

@MikeEdgar MikeEdgar requested a review from a team August 28, 2024 19:14
@MikeEdgar
Copy link
Member

The storybook tests don't seem to be picking up the change for the added div.

@MikeEdgar MikeEdgar force-pushed the dependabot/npm_and_yarn/ui/patternfly-be2d9dd4ed branch from a0e7a1b to c920b98 Compare August 29, 2024 17:33
@MikeEdgar
Copy link
Member

The storybook tests don't seem to be picking up the change for the added div.

Even though the component works fine in the prod app, storybook does not provide an element in the DOM with id="root". We won't be able to update patternfly unless we can work around it or propose some other fix in patternfly instead of patternfly/patternfly-react#10283

@hemahg
Copy link
Contributor

hemahg commented Aug 29, 2024

@MikeEdgar, I also tried some workarounds but still storybook is not working for Dragdrop, I will discuss this issue in the Patternfly slack channel

@riccardo-forina
Copy link
Collaborator

Have you tried adding the div like this?

dependabot bot and others added 3 commits September 3, 2024 11:10
Bumps the patternfly group in /ui with 7 updates:

| Package | From | To |
| --- | --- | --- |
| [@patternfly/patternfly](https://github.com/patternfly/patternfly) | `5.3.1` | `5.4.0` |
| [@patternfly/react-charts](https://github.com/patternfly/patternfly-react) | `7.3.0` | `7.4.0` |
| [@patternfly/react-core](https://github.com/patternfly/patternfly-react) | `5.3.4` | `5.4.0` |
| [@patternfly/react-drag-drop](https://github.com/patternfly/patternfly-react) | `5.3.4` | `5.4.0` |
| [@patternfly/react-icons](https://github.com/patternfly/patternfly-react) | `5.3.2` | `5.4.0` |
| [@patternfly/react-table](https://github.com/patternfly/patternfly-react) | `5.3.4` | `5.4.0` |
| [@patternfly/react-tokens](https://github.com/patternfly/patternfly-react) | `5.3.1` | `5.4.0` |


Updates `@patternfly/patternfly` from 5.3.1 to 5.4.0
- [Release notes](https://github.com/patternfly/patternfly/releases)
- [Changelog](https://github.com/patternfly/patternfly/blob/main/release.config.js)
- [Commits](patternfly/patternfly@patch-v5.3.1...v5.4.0)

Updates `@patternfly/react-charts` from 7.3.0 to 7.4.0
- [Release notes](https://github.com/patternfly/patternfly-react/releases)
- [Commits](https://github.com/patternfly/patternfly-react/compare/@patternfly/[email protected]...@patternfly/[email protected])

Updates `@patternfly/react-core` from 5.3.4 to 5.4.0
- [Release notes](https://github.com/patternfly/patternfly-react/releases)
- [Commits](https://github.com/patternfly/patternfly-react/compare/@patternfly/[email protected]...@patternfly/[email protected])

Updates `@patternfly/react-drag-drop` from 5.3.4 to 5.4.0
- [Release notes](https://github.com/patternfly/patternfly-react/releases)
- [Commits](https://github.com/patternfly/patternfly-react/compare/@patternfly/[email protected]...@patternfly/[email protected])

Updates `@patternfly/react-icons` from 5.3.2 to 5.4.0
- [Release notes](https://github.com/patternfly/patternfly-react/releases)
- [Commits](https://github.com/patternfly/patternfly-react/compare/@patternfly/[email protected]...@patternfly/[email protected])

Updates `@patternfly/react-table` from 5.3.4 to 5.4.0
- [Release notes](https://github.com/patternfly/patternfly-react/releases)
- [Commits](https://github.com/patternfly/patternfly-react/compare/@patternfly/[email protected]...@patternfly/[email protected])

Updates `@patternfly/react-tokens` from 5.3.1 to 5.4.0
- [Release notes](https://github.com/patternfly/patternfly-react/releases)
- [Commits](https://github.com/patternfly/patternfly-react/compare/@patternfly/[email protected]...@patternfly/[email protected])

---
updated-dependencies:
- dependency-name: "@patternfly/patternfly"
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: patternfly
- dependency-name: "@patternfly/react-charts"
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: patternfly
- dependency-name: "@patternfly/react-core"
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: patternfly
- dependency-name: "@patternfly/react-drag-drop"
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: patternfly
- dependency-name: "@patternfly/react-icons"
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: patternfly
- dependency-name: "@patternfly/react-table"
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: patternfly
- dependency-name: "@patternfly/react-tokens"
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: patternfly
...

Signed-off-by: dependabot[bot] <[email protected]>
Signed-off-by: Michael Edgar <[email protected]>
@MikeEdgar
Copy link
Member

Have you tried adding the div like this?

Nope, I'll try that now.

@MikeEdgar MikeEdgar force-pushed the dependabot/npm_and_yarn/ui/patternfly-be2d9dd4ed branch from c920b98 to 7151416 Compare September 3, 2024 15:12
Copy link

sonarcloud bot commented Sep 3, 2024

@riccardo-forina
Copy link
Collaborator

It looks like that did the trick :)

@MikeEdgar MikeEdgar merged commit 9578275 into main Sep 3, 2024
9 checks passed
@MikeEdgar MikeEdgar deleted the dependabot/npm_and_yarn/ui/patternfly-be2d9dd4ed branch September 3, 2024 15:38
@MikeEdgar MikeEdgar added this to the 0.3.4 milestone Sep 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file javascript Pull requests that update Javascript code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants