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

docs(react-component-groups): Creates overview page for extension. #81

Merged
merged 6 commits into from
Nov 21, 2023

Conversation

edonehoo
Copy link
Collaborator

Closes #79

Copy link
Collaborator

@dlabaj dlabaj left a comment

Choose a reason for hiding this comment

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

A few comments but over all looks good.

Copy link
Contributor

@jessiehuff jessiehuff left a comment

Choose a reason for hiding this comment

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

This looks good so far, but it seems a little bare at the moment. Some other things that could be good to add are:

  • Some kind of getting started with this repo - what are some of the basics that a user would have to know in order to use component groups? The extensions that do have something similar to an About page typically help give the user general background info on how to use it. Ex: Quick Starts and Topology (@dlabaj maybe you can help suggest something for this? Doesn't have to be anything super complex, just wondering what are the basics a user needs to know to get started using it.)
  • Contributing to the repo - How do others contribute to the repo? Perhaps we can make a note about where to go to in order to create issues. Do we have any expectations for those wanting to contribute?
  • Standards of this repo - since this extension is one of the first, genuine collaborations with a team when it comes to contributions, I'm wondering if perhaps we should make some mention of the standards. (Other extensions were often pulled from products as well, but PF took full control once it was an extension. This extension has felt far more collaborative as we've been working closely with HCC. Because of that, we're trying to be a little flexible but still hold it to PF standards.)

Copy link
Collaborator

@dlabaj dlabaj left a comment

Choose a reason for hiding this comment

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

Added a suggestion on direct imports. Feel free to word smith it.


The **patternfly/react-component-groups** extension contains a range of React components that are more complex than basic PatternFly components, often combining and adjusting multiple base components to address common use cases.

This extension replaces **RedHatInsights/frontend-components**. To migrate your product to this new implementation, you can refer to migration guide details in [the README in the React component groups repository](https://github.com/patternfly/react-component-groups/migration.md).
Copy link
Collaborator

@dlabaj dlabaj Nov 21, 2023

Choose a reason for hiding this comment

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

We could add the following describing imports.

Direct Imports

React component groups supports direct imports. This is useful in improving your bundle size. To directly import a component your import should look similar to the following:
import Ansible from '@patternfly/react-component-groups/dist/dynamic/Ansible';

Alternatively could use a plugin to change relative imports to direct imports. Direct imports are specifically important when using module federation in webpack. To learn more about the importance importing has on your bundle size see the following article:

Webpack module federation think twice before sharing a dependency

@nicolethoen
Copy link
Contributor

I think at the top of this page, it should make it clear that the components in this section are coming from the @patternfly/react-component-groups package and link to this NPM page: https://www.npmjs.com/package/@patternfly/react-component-groups

It should also highlight that the code for these components lives in the following repo: https://github.com/patternfly/react-component-groups
So that is where issues should be filed for these components.

@edonehoo
Copy link
Collaborator Author

Does the "sortValue" param work at this nav level?

@dlabaj dlabaj merged commit c59bfef into patternfly:main Nov 21, 2023
4 checks passed
Copy link

🎉 This PR is included in version 5.0.0-prerelease.10 🎉

The release is available on:

Your semantic-release bot 📦🚀

@nicolethoen
Copy link
Contributor

yeah - adding sortValue: 1 at the top of the markdown file as you have it here should work.
@edonehoo did it look like it was working to you?

@edonehoo
Copy link
Collaborator Author

@nicolethoen I haven't able to get my local build to work for this repo, so I'm not sure 😅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update docs to include background
4 participants