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

Individual module imports for navigation components #393

Merged
merged 1 commit into from Jun 17, 2018

Conversation

dlabrecq
Copy link
Member

Individual module imports for navigation components.

This allows components to be imported individually without having to install an unused, optional dependency.

Deprecations

  • NavigationItemConfig: Use NavigationBaseConfig, VerticalNavigationConfig, or ApplicationLauncherConfig
  • NavigationModule: Use ApplicationLauncherModule or VerticalNavigationModule

Note: Backed out #377 to address a git commit issue with Travis. Adding code back in one module at a time.

This allows components to be imported individually without having to install an unused, optional dependency.

Deprecations

- NavigationItemConfig: Use NavigationBaseConfig, VerticalNavigationConfig, or ApplicationLauncherConfig
- NavigationModule: Use ApplicationLauncherModule or VerticalNavigationModule
@dlabrecq dlabrecq merged commit 8a08dfe into patternfly:master Jun 17, 2018
dlabrecq added a commit that referenced this pull request Jun 18, 2018
)

This allows components to be imported individually without having to install an unused, optional dependency.

Deprecations

- NavigationItemConfig: Use VerticalNavigationItem or ApplicationLauncherItem
- NavigationModule: Use ApplicationLauncherModule or VerticalNavigationModule
dlabrecq added a commit to dlabrecq/patternfly-ng that referenced this pull request Jun 18, 2018
…atternfly#393)

This allows components to be imported individually without having to install an unused, optional dependency.

Deprecations

- NavigationItemConfig: Use VerticalNavigationItem or ApplicationLauncherItem
- NavigationModule: Use ApplicationLauncherModule or VerticalNavigationModule
@dlabrecq dlabrecq deleted the modules-navigation branch June 18, 2018 14:44
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.

None yet

1 participant