Closed (fixed)
Project:
Flashy
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
20 Aug 2009 at 22:20 UTC
Updated:
8 Sep 2009 at 22:40 UTC
Here's a patch to enable users to install the videoplayer file through drush instead of having to download it manually.
| Comment | File | Size | Author |
|---|---|---|---|
| flashy-drush.patch | 1.72 KB | eabrand |
Comments
Comment #1
moshe weitzman commentedgood idea. found minor points ...
missing command help. just a brief one liner.
I don't think SIMULATE should say that. it is lying.
minor ... use drush_log("Downloading $file was successful to $filename.", 'success'). this is better for remotely executed commands
strings should be emitted inside of dt(). thats the drush equivalent of t()
This review is powered by Dreditor.
Comment #2
Crell commentedAlso, after some discussion with Moshe in IRC we decided that "post-install" was probably a better command name standard than "install".
Comment #3
Crell commentedCommitted with the comments from #1 and #2 addressed. Yay!