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

Install only specific flags from react-flagpack #77

Open
ernstoarllano opened this issue Aug 26, 2024 · 1 comment
Open

Install only specific flags from react-flagpack #77

ernstoarllano opened this issue Aug 26, 2024 · 1 comment

Comments

@ernstoarllano
Copy link

Detailed Description

Currently, when using the react-flagpack library in a Next.js project, all SVGs are installed in the public folder by default. This request proposes a way to selectively install only the specific flags needed, rather than the entire collection.

Is your feature request related to a problem? Please describe.

The default installation of all SVGs can lead to unnecessary bloat in the project. It would be more efficient to install only the flags that are needed.

Describe the solution you'd like

A feature that allows users to specify which flags to include during the installation process, reducing the overall size of the package in the project.

Describe alternatives you've considered

Manually deleting unused SVGs from the public folder post-installation, but this is not an ideal solution.

Additional context

This feature would help minimize unnecessary bloat in the project, particularly for cases where only a limited number of flags are needed. By allowing selective installation, the overall efficiency of the development process can be improved, keeping the project lean and optimized.

@Yummygum Yummygum deleted a comment from dseynhae Sep 25, 2024
@zoeyfrisart
Copy link
Member

Hi there,
First of all, apologies for the malicious content shared in the comment above.
This comment slipped through, and we didn't notice that someone had replied. Please don't download the link in the comment.

We appreciate the suggestion, and it is something we are considering. However, this does require some reworking of the CLI. We plan to add this in the future, but unfortunately, we don't have the resources to do so now. If someone wants to create a pull request for this, give me a tag and I'll have a look at it.

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

No branches or pull requests

2 participants