search

terminus-plugin-project / terminus-pancakes-plugin

Terminus Plugin to open Pantheon Site Databases in your Favorite SQL Client
MIT License
26 stars 20 forks source link

Fix pantheon site URL output in logs #39

Open incredimike opened 6 months ago

incredimike commented 6 months ago

This PR is a simple fix to logging output.

Currently, the site environment hostname output to console is incorrect. See example below:

$ terminus pc ucf-sdes.dev
[notice] Multiple Pancakes Applications were found: Sequel Pro, SequelAce, MySQL. Add --app to be specific on the app.
[notice] Opening dev-ucf-sdes.pantheon.io database in Sequel Pro.
$

The host should be: dev-ucf-sdes.pantheon.io. It's missing the string "site" in the output.

I also noticed around line 50 in PancakesCommand.php the same string .pantheon.io is used when creating some IDs, but but those IDs probably don't need to change(?) so I left it.