Skip to content
This repository has been archived by the owner on Sep 4, 2020. It is now read-only.

Uppercase names in CSV have unexpected results #9

Open
davidneedham opened this issue Nov 1, 2018 · 1 comment
Open

Uppercase names in CSV have unexpected results #9

davidneedham opened this issue Nov 1, 2018 · 1 comment

Comments

@davidneedham
Copy link
Contributor

davidneedham commented Nov 1, 2018

Terminus/Pantheon converts uppercase characters in the machine name to lowercase. If this script is referring to a CSV with uppercase student names, it will create a machine name with uppercase characters. Pantheon doesn't mind and continues along, but any future commands run against the uppercase version fails because the site doesn't exist.

You can simulate this by doing terminus site:create JDoe-my-example "JDoe-my-example" WordPress. When it's done, try terminus site:delete JDoe-my-example. It will say that you don't have access to that site or it doesn't exist. But terminus site:delete jdoe-my-example works.

As a result, every Terminus command run after the site is created fails with an error saying you don't have access.

I propose that the script make all machine names lowercase.

@ataylorme
Copy link
Owner

I propose that the script make all machine names lowercase.

As a workaround in the meantime can you make the names in the spreadsheet lowercase? Also this seems like a request we might want to make upstream to Terminus so that Terminus itself takes machine name input and makes them lowercase before processing.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants