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

chore(upgrade): upgrade deps, fix storybook #103

Merged
merged 1 commit into from
Jan 26, 2021

Conversation

seanforyou23
Copy link
Collaborator

@seanforyou23 seanforyou23 commented Jan 6, 2021

This PR upgrades runtime and development dependencies to latest version and puts storybook back in to a semi-functional state. Storybook still seems to have issues with webpack 5. Currently storybook is completely broken, but with this PR it at least boots up. That said, when you navigate to the Support component story you get the following error. I spent a bit of time looking at it but didn't find a solution just yet. Figured I'd post this PR for review in the mean time in case someone else knows what gives.

Screen Shot 2021-01-06 at 1 18 35 PM

http://patternfly-react-seed.surge.sh/
http://patternfly-react-seed-storybook.surge.sh/

Steps toward fixing #100
Invalidates #65

use localhost instead of 0.0.0.0 for dev hostname so app opens correctly on windows
use enzyme-adapter-react-17 to ensure jest test works
remove stub async module support
prevent url from changing when selecting SkipToContent component
update readme screenshot preview

use localhost instead of 0.0.0.0 for host so app opens correctly on windows
use enzyme-adapter-react-17 to ensure jest test works
remove stub async module support
prevent url from changing when selecting SkipToContent component
update readme screenshot preview
patch storybook
@seanforyou23 seanforyou23 merged commit c612b7b into patternfly:master Jan 26, 2021
bhanvimenghani pushed a commit to bhanvimenghani/patternfly-react-seed that referenced this pull request Mar 18, 2022
use localhost instead of 0.0.0.0 for host so app opens correctly on windows
use enzyme-adapter-react-17 to ensure jest test works
remove stub async module support
prevent url from changing when selecting SkipToContent component
update readme screenshot preview
patch storybook
@mturley
Copy link
Collaborator

mturley commented Aug 19, 2022

Note for posterity: since this PR removed the stub async route support (and the DynamicImport file orphaned here was later removed in #108) but the isAsync property remains (and is still used to exclude routes from the accessible focus handler introduced in #57), that focus handler fails to work when navigating to the Support route which was previously async. I discovered this as part of attempting to remove our dependency on react-router-last-location in an effort to fix #134. I'll complete the cleanup from this change in the PR to fix that issue.

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