silverstripe / bambusa-installer

SilverStripe demo application
https://silverstripe.com/demo
BSD 3-Clause "New" or "Revised" License
11 stars 8 forks source link

Demo creation fails if site name does not start with a letter character (incl CWP) #111

Closed brynwhyman closed 4 years ago

brynwhyman commented 4 years ago

Example error message: (value is \"6xpertscom-sscom\"): only lowercase letters, numbers, underscores and dashes are allowed, must start with a letter"

This is a limitation of the underlying demo creation infrastructure. We see the same problem with the CWP demo: https://github.com/silverstripeltd/cwp-demo/issues/18

It seems that a quick win here would be some validation on the demo request from.

There's a related issue that could also benefit from front-end validation: https://github.com/silverstripe/bambusa-installer/issues/87

AC's

PR's

maxime-rainville commented 4 years ago

We're adding a suffix to the demo name based on what environment the demo is created on. This is meant to avoid collisions between CWP and DotCOM demoes.

A simple fix could be to make it a prefix instead of a suffix.

brynwhyman commented 4 years ago

We've got the same problem with the CWP demo so it seems we should do the same fix for both. I don't believe the CWP demos have a prefix.

If we fix this with form validation, we could probably squeeze the following in as well: https://github.com/silverstripe/bambusa-installer/issues/87

bergice commented 4 years ago

Added new PR's to update sites with merged demostacker changes and to fix circleCI builds on silverstripe.com.

brynwhyman commented 4 years ago

Blocked by: https://silverstripe.freshdesk.com/a/tickets/67189

The latest suggestion is to do a re-deploy of the base stack holding the ss.com vstack. This could result in a PHP upgrade for other projects, so we'll need to make sure that has no adverse affects. I can have a look at this and track someone down to help approve that.

brynwhyman commented 4 years ago

We've received approval from Olivia to proceed with a base stack deployment: https://silverstripeltd.slack.com/archives/C0XL54DEH/p1598910385007000?thread_ts=1598910107.006700&cid=C0XL54DEH