Skip to content

Add installer for Processwire module #472

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.

Sign up for GitHub

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

Jump to bottom
Merged
niksamokhvalov merged 4 commits into from
Jan 12, 2021
Merged

Add installer for Processwire module #472

niksamokhvalov merged 4 commits into from
Jan 12, 2021

Conversation

@daun
Copy link
Contributor

@daun daun commented Dec 9, 2020 •
edited
Loading

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 daun mentioned this pull request Dec 9, 2020
Closed
@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

niksamokhvalov commented Jan 12, 2021

Yes! In the next few days.

@tiefenb tiefenb mentioned this pull request Jun 28, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@daun @niksamokhvalov