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

Add installer for Processwire module #472

Merged
merged 4 commits into from
Jan 12, 2021

Conversation

daun
Copy link
Contributor

@daun daun commented Dec 9, 2020

Install CamelCased dependencies into site/modules/.

For easier review, this is what the official docs about ProcessWire modules have to say:

Site modules (stored in /site/modules/) are those that you obtain separately and install yourself.

Installing a site module is as simple as uploading the module's files to the /site/modules/ModuleName/ directory on your server (where ModuleName is the name of the module).

@daun
Copy link
Contributor Author

daun commented Dec 19, 2020

What's the best way to get this reviewed and merged? @naderman @Seldaek

Sorry for spamming you directly, just not sure about the review process for new installers. I added relevant documentation in the PR comment to help along the process, but let me know if it's missing any crucial information.

@niksamokhvalov niksamokhvalov merged commit 9d4e028 into composer:main Jan 12, 2021
@daun
Copy link
Contributor Author

daun commented Jan 12, 2021

Thanks 🌝 Any chance to get a new tagged release @niksamokhvalov?

@niksamokhvalov
Copy link
Member

Yes! In the next few days.

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.

2 participants