-
-
Notifications
You must be signed in to change notification settings - Fork 32
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
Updated the script to support installing multiple Flarum extensions in one command. #106
base: master
Are you sure you want to change the base?
Conversation
Updated the script to support installing multiple Flarum extensions in one command.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Small nits to fix otherwise looks good, thanks!
Also need to update the README:
Lines 191 to 195 in a32f450
`docker compose exec flarum extension require <package>` | |
To remove an extension: | |
`docker compose exec flarum extension remove <package>` |
Can you squash your commits as well when fixed please?
Co-authored-by: CrazyMax <1951866+crazy-max@users.noreply.github.com>
Update CHANGELOG Update extension Updated the script to support installing multiple Flarum extensions in one command. Apply suggestions from code review Co-authored-by: CrazyMax <1951866+crazy-max@users.noreply.github.com> new year update README update README Flarum 1.8.5 Update CHANGELOG update readme.md
I'm relatively new to Git and have done my best to follow the online guide in this pull request. However, if there are any inaccuracies or issues, please feel free to make adjustments or guide me on how to correct them. I appreciate your understanding and help! |
While migrating from a direct installation to your Docker version, I faced several issues, one being the incorrect migration of plugins. I resolved this by reinstalling them, requiring a script to install over seventy plugins. Recently, I discovered that modifying the script could expedite this process. Therefore, I had ChatGPT modify the code to support processing multiple package names simultaneously.