-
Notifications
You must be signed in to change notification settings - Fork 60
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
islandora_install_profile_demo missing #318
Comments
Sorry in advance for the wall of text. The name would need to be changed in the islandora install profile demo and islandora sandbox https://github.com/Islandora-Devops/islandora_install_profile_demo/blob/main/composer.json#L2 I will add though that the url for that repo is pointing to the right github repository in islandora sandbox https://github.com/Islandora-Devops/islandora-sandbox/blob/main/composer.json#L18 The islandora name is also the same for islandora-sandbox, islandora_base_theme, and islandora_starter_theme repos https://github.com/Islandora-Devops/islandora-sandbox/blob/main/composer.json#L2 Are these suppose to be named islandora-devops instead of islandora? If that's the correct vendor, there would need to be changes in 4 different repositories' composer.json file and islandora sandbox's composer.lock file for both the name of the package and the name in the require section for the other repositories with name changes that it references. |
I see - the name here is cosmetic - it could be anything. @aOelschlager got it exactly right. If we want to change this cosmetic name to be I don't have a strong opinion on this. I think it's fine as-is, but maybe others disagree. |
Thanks. I'll close |
docker-compose exec -T drupal with-contenv bash -lc "drush si -y islandora_install_profile_demo --account-pass password"
is pointing to "islandora\islandora_install_profile_demo" This is pointing to the wrong location.
It should point to https://github.com/Islandora-Devops/islandora_install_profile_demo
This needs to be added to one of the repose that brings in this repo
The text was updated successfully, but these errors were encountered: