Updated Sequel Pro addon to use safe project name

docksal / addons

Community driven, submit your addon! Docksal addons that can be installed via `fin addon install` command.

MIT License

16 stars 34 forks source link

Updated Sequel Pro addon to use safe project name #46

Closed moacode closed 5 years ago

moacode commented 5 years ago

Fixes an issue where project names containing symbols caused the port to not be found, resulting in a Sequel Pro error.

moacode commented 5 years ago

We often name our repositories after the domain name i.e. mysite.co.nz. The Docker filter being run in the addon script to determine the port number would fail due to Docker removing the periods in the container name. Using the $COMPOSE_PROJECT_NAME_SAFE variable resolves this issue.

lmakarov commented 5 years ago

Thanks!