Dovetailing off #1363240: Bravo! plus a small need for docs, which I found looking for this feature request, I think it would be helpful to have a quick dsm() with instructions to add remote host to settings.php. I love this module, and I use it every time I have a new project, but it sure is nice to have something to copy/paste into settings.php to get rolling.
Patch coming once I post and have an issue queue to name with.
Love the site-alias bit as well, but not sure: is it only for after sql-sync, or is this the same as adding to settings.php?
Thanks for your contributions and consideration!
-Bronius
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | add-hook-enable-config-message-1871716.patch | 538 bytes | texas-bronius |
Comments
Comment #1
texas-bronius commentedPatch to provide requested feature:
* added .install and a hook_enable with message to include minimal configuration to get up and running
Comment #2
gregglesSeems like a decent idea, depending on how #1014466: Admin interface goes.
Comment #3
texas-bronius commentedAh- Thanks for crosslinking the threads. I don't see why they couldn't both live together (with modifications to this should #1014466: Admin interface bake in).
In my use case, I am usually quickly throwing together a dev env for troublehsooting or starting dev, and I live in drush as much as possible/needed. A quick blurb reminder is helpful enough for me and would be useful even if an admin UI were made available.
Comment #4
texas-bronius commentedOh I see why you might hold off on this. WIth the linked ticket above, you're putting the variable in admin UI.
Well, I think if you're keeping the ability to set it in settings.php, there's no harm in putting in a hook_enable message: Easy UX improvement for the developer and easier adoption to the drush user new to the module.
If you want to get geek with it, add something like, "Or, at minimum, use vset stage_file_proxy_origin http://example.com //(no trailing slash)" to the message :)
Comment #5
gregglesThe project page and README now have drush instructions. I think this should be skipped.