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

Update packagist name and convert to vendormodule #38

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

elliot-sawyer
Copy link

@elliot-sawyer elliot-sawyer commented Aug 26, 2019

The packagist name for this repository seems to have changed. If you google "silverstripe maintenance mode" and click on the addons page, it points at the following URLs along with this repository:
Addons:
https://addons.silverstripe.org/add-ons/thisisbd/silverstripe-maintenance-mode
Homepage:
http://github.com/dljoseph/silverstripe-maintenance-mode
Packagist:
https://packagist.org/packages/thisisbd/silverstripe-maintenance-mode
Repository:
https://github.com/dljoseph/silverstripe-maintenance-mode

Changing the package name allows the module to be installed via composer. Changing it to vendormodule allows it to be installed in the vendors folder (see issue #31 )

The packagist name for this repository seems to have changed. If you google "silverstripe maintenance mode" and click on the addons page, it points at the following URLs:
Addons:
    https://addons.silverstripe.org/add-ons/thisisbd/silverstripe-maintenance-mode
Homepage:
    http://github.com/dljoseph/silverstripe-maintenance-mode 
Packagist:
    https://packagist.org/packages/thisisbd/silverstripe-maintenance-mode
Repository:
    https://github.com/dljoseph/silverstripe-maintenance-mode

Changing the package name allows the module to be installed via composer. Changing it to vendormodule allows it to be installed in the vendors folder (see issue dynamic#31 )
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.

1 participant