These are chat archives for internalfx/quickshot

19th
Mar 2015
Bryan Morris
@internalfx
Mar 19 2015 01:55
@rickydazla it's working!
You can configure as many "targets" as you want
If you have only 1 target it works as before
If you have muliple targets then it asks you which one you want to use when downloading/uploading/watching
a target = a specific theme at a specific shopify shop
You can give them a name as well like "dev" or "prod" whatever you want
package is released to NPM as quickshot@0.4.0
Bryan Morris
@internalfx
Mar 19 2015 02:03
working on docs now
Bryan Morris
@internalfx
Mar 19 2015 02:58
Docs updated
Rick Davies
@rickydazla
Mar 19 2015 15:41
sweet, i'll give it a shot today
Rick Davies
@rickydazla
Mar 19 2015 16:39
On download: Assertion failed: (end <= source_len), function SliceOnto, file ../src/smalloc.cc, line 280.
Abort trap: 6
Although up to that point I see it was grabbing files quicker than hell!
Bryan Morris
@internalfx
Mar 19 2015 17:32
You can also upload a subset of files by passing a filter
quickshot upload assets
Will only upload the assets folder
Rick Davies
@rickydazla
Mar 19 2015 17:33
Yes I am experimenting, it’s really nice
Bryan Morris
@internalfx
Mar 19 2015 17:33
file ../src/smalloc.cc, line 280. Abort trap: 6
Not sure what to think of that, clearly a problem in a C library...But I dont write C :worried:
Rick Davies
@rickydazla
Mar 19 2015 17:34
haha ok no worries
I am looking at the theme selection..
You list them A-Z but no indicator of which is primary / live
Bryan Morris
@internalfx
Mar 19 2015 17:35
Yeah, it grabs all the available themes from the shop...no more hunting for theme ids!
Rick Davies
@rickydazla
Mar 19 2015 17:35
Also, in theme gem if you leave theme_id blank it defaults to whatever is live
Bryan Morris
@internalfx
Mar 19 2015 17:35
ooooo...good point
I do require a theme_id...It's risky without I think. I think its best if users are explicit about what they want quickshot to do...
I dont want to take down anyones live site
Do most people not name themes like "whatever name - live" or somethings
something*
Rick Davies
@rickydazla
Mar 19 2015 17:37
I have no idea!
I’m just thinking that “quick requests" I often deal with just require I download/edit/modify one file from the live theme. With qs config as is I have no way to tell which is live theme without logging in to admin panel
Bryan Morris
@internalfx
Mar 19 2015 17:40
Gotcha
Rick Davies
@rickydazla
Mar 19 2015 17:40
In one store right now I see themes with labels V3, V3_responsive, V3_optimized
I was not sure which was live… turned out to be V3_responsive
With theme gem and no ID specified it does not matter - I am just pulling file from whatever is live
Bryan Morris
@internalfx
Mar 19 2015 17:43
Its a good idea, mind making an issue?
Rick Davies
@rickydazla
Mar 19 2015 17:45
no worries
Thanks for making and publishing this + being willing to listen btw
Bryan Morris
@internalfx
Mar 19 2015 18:06
Your Welcome, I feel like there is no real solid "goto" solution for working locally with Shopify themes.
I hope quickshot can become the defacto standard, with features designers really enjoy.
Rick Davies
@rickydazla
Mar 19 2015 18:07
It will be my defacto as soon as I get all my config files set up!
Bryan Morris
@internalfx
Mar 19 2015 18:07
Adoption and feedback are helpful, I'm not a designer, so I need feedback from them.
How much faster is it than shopify_theme?
Rick Davies
@rickydazla
Mar 19 2015 18:08
a lot
!
Bryan Morris
@internalfx
Mar 19 2015 18:08
Cool, shopify_theme wasnt really pushing the available API limits....
quickshot will upload/download nearly ~40 files at a time
Rick Davies
@rickydazla
Mar 19 2015 18:09
Do you use any terminal package(s) with atom?
Bryan Morris
@internalfx
Mar 19 2015 18:09
then it backs off to respect the API
terminal packages?
Rick Davies
@rickydazla
Mar 19 2015 18:10
but the arrow key nav thing within quickshot configure does not seem to work with that
Bryan Morris
@internalfx
Mar 19 2015 18:10
Oh I see
I havent, used any in atom...they didnt work well for me..
I assumed it was a Linux thing
But they may have been much more alpha at the time
Hmmm....I may have to hunt for a workaround
Rick Davies
@rickydazla
Mar 19 2015 18:12
Can I specify quickshot watch --staging or similar?
Bryan Morris
@internalfx
Mar 19 2015 18:13
Being that quickshot is for designers I wanted the menus to be easy to navigate, without needing arcane commands and such
Rick Davies
@rickydazla
Mar 19 2015 18:13
both? :cake:
Bryan Morris
@internalfx
Mar 19 2015 18:13
I have plans to add the ability to specify the target as an argument
Rick Davies
@rickydazla
Mar 19 2015 18:13
yummy
Bryan Morris
@internalfx
Mar 19 2015 18:14
but its not there yet, you have to select it each time
Bryan Morris
@internalfx
Mar 19 2015 18:31
npm update -g quickshot
Rick Davies
@rickydazla
Mar 19 2015 19:37
how do I access configuration to reconfigure?
Bryan Morris
@internalfx
Mar 19 2015 19:37
quickshot configure
You can run it again anytime....
Rick Davies
@rickydazla
Mar 19 2015 19:38
creates a new one?
Bryan Morris
@internalfx
Mar 19 2015 19:38
Nope...You are gonna like it when you try it though!!!
Rick Davies
@rickydazla
Mar 19 2015 19:39
It is asking me to create a target
I want to edit an existin one
I can see the json file obv:)
my bad! I got it
Bryan Morris
@internalfx
Mar 19 2015 19:48
It does have to be run from the location of the quickshot.json file
You can alter your config at any time
its designed to never need to be edited by hand
Rick Davies
@rickydazla
Mar 19 2015 19:50
yes, I was in the module dir, had to re-install iced-runtime module when I updated
Bryan Morris
@internalfx
Mar 19 2015 19:52
oops, your right...that's a missing dependency...
fixed
published
Bryan Morris
@internalfx
Mar 19 2015 20:08
@rickydazla If your buddies are having trouble have them run npm update -g quickshot
current version is 0.5.1
Rick Davies
@rickydazla
Mar 19 2015 21:22
I think missing from docs are the commands. I know they are in--help but without knowing that command you would not find them
e.g. you haven’t mentioned quickshot watch to actually get it going
Bryan Morris
@internalfx
Mar 19 2015 21:23
Right...I'm sure the docs could use some work.
Rick Davies
@rickydazla
Mar 19 2015 21:23
Perhaps between “The configuration wizard” and “Autocompiling css"
Thomas Reggi
@reggi
Mar 19 2015 21:24
Hey @internalfx and @rickydazla
Rick Davies
@rickydazla
Mar 19 2015 21:24
Happy to help with that if you want?
Thomas Reggi
@reggi
Mar 19 2015 21:24
Just installed
Bryan Morris
@internalfx
Mar 19 2015 21:24
Docs are gonna be important, as this is not meant for people who want to tear apart the code
That would be great
Thomas Reggi
@reggi
Mar 19 2015 21:24
I think that this should be in the readme somewhere
  quickshot configure              Create a new configuration file in current directory
  quickshot download [filter]      Download theme files, optionally providing a filter
  quickshot upload [filter]        Upload theme files, optionally providing a filter
  quickshot watch                  Watch project folder and synchronize changes automatically
  quickshot --help                 Show this screen.
Rick Davies
@rickydazla
Mar 19 2015 21:24
Can write docs, cannot tear appart code : )
Bryan Morris
@internalfx
Mar 19 2015 21:24
Hey @reggi we were just talking about that
Thomas Reggi
@reggi
Mar 19 2015 21:24
:)
Bryan Morris
@internalfx
Mar 19 2015 21:24
Oh...just noticed thats wrong
quickshot configure Create a new configuration file in current directory
it also updates configuration
Rick Davies
@rickydazla
Mar 19 2015 21:25
yeah right, that’s what confused me earlier
Bryan Morris
@internalfx
Mar 19 2015 21:25
I want to make it clear that quickshot configure also changes configuration
Thomas Reggi
@reggi
Mar 19 2015 21:26
Now, my project folder has a theme folder which has my shopify theme in it. Do I have to nest everything theme because that's gonna be a mess cd ./theme && quickshot watch everytime
Bryan Morris
@internalfx
Mar 19 2015 21:27
right now quickshot works directly in the theme folder
Thomas Reggi
@reggi
Mar 19 2015 21:27
so I have to have the config in the theme folder
and run quickshot from my theme
Bryan Morris
@internalfx
Mar 19 2015 21:27
but a lot of design choices are up in the air at this point
yeah
Thomas Reggi
@reggi
Mar 19 2015 21:27
:/
it's ok not too bad
Now I have to ensure one last thing.
I have a really complicated setup where I can't upload the whole theme, the watch script it's gonna upload only changed files right?
Bryan Morris
@internalfx
Mar 19 2015 21:28
yes
as you change them
Thomas Reggi
@reggi
Mar 19 2015 21:28
Does it notify on mac osx?
good
Bryan Morris
@internalfx
Mar 19 2015 21:29
It uses a mac compatible library, but currently uses polling.
Our in-house designer uses it
Rick Davies
@rickydazla
Mar 19 2015 21:29
I have a themes folder with 140 themes in it but v simple set up. scss compiled by shopify :)
Bryan Morris
@internalfx
Mar 19 2015 21:29
hes on a mac seems to work great for him
You can certainly let shopify do the scss
The reason I added local compiling is because you get more power...
@imports especially
Your site gets a performance boost by having fewer files to fetch from the web
Rick Davies
@rickydazla
Mar 19 2015 21:31
sure, i know. this would certainly help with that. to this point would just have been too much config for me to be arsed. many of those themes are still using vanilla css
Bryan Morris
@internalfx
Mar 19 2015 21:31
you could even compile all of bootstrap into your regular styles with quickshot
Thomas Reggi
@reggi
Mar 19 2015 21:31
@internalfx does it support deleting files?
Bryan Morris
@internalfx
Mar 19 2015 21:32
if you are currently watching, yes
Thomas Reggi
@reggi
Mar 19 2015 21:32
Ok that's cool
:)
:shipit:
So there's no notifications
Can we have an option for terminal notifier? https://github.com/alloy/terminal-notifier
Bryan Morris
@internalfx
Mar 19 2015 21:33
no notifications?
Thomas Reggi
@reggi
Mar 19 2015 21:34
It's bash notifications but not growl-like notifications
Bryan Morris
@internalfx
Mar 19 2015 21:34
it should print to the terminal what its doing :confused:
Thomas Reggi
@reggi
Mar 19 2015 21:34
it does
Bryan Morris
@internalfx
Mar 19 2015 21:34
oh like OS notifications?
Thomas Reggi
@reggi
Mar 19 2015 21:34
yeah
Bryan Morris
@internalfx
Mar 19 2015 21:34
like from chat programs and such?
Thomas Reggi
@reggi
Mar 19 2015 21:34
yep
Rick Davies
@rickydazla
Mar 19 2015 21:35
Screen Shot 2015-03-19 at 14.34.39.png
Thomas Reggi
@reggi
Mar 19 2015 21:35
that's what this does
terminal-notifier -title Shopify -sound Tink -message "Shopify theme being watch" -group shopify
cd ./theme &&
theme watch |
sed -E -l -e 's/.+ Uploaded: (.+)/\1/' |
xargs -I output terminal-notifier -title Shopify -sound Tink -message output -group shopify
Rick Davies
@rickydazla
Mar 19 2015 21:35
is nice colours
Bryan Morris
@internalfx
Mar 19 2015 21:36
Yeah, I didnt even know about the shopify pages...
is that your pages code in red?
Thomas Reggi
@reggi
Mar 19 2015 21:37
What is that pages
Bryan Morris
@internalfx
Mar 19 2015 21:37
The assets API will reject files that are not in the correct folders
Thomas Reggi
@reggi
Mar 19 2015 21:37
you can't have a pages directory in the theme
Bryan Morris
@internalfx
Mar 19 2015 21:37
I think that may just be where @rickydazla stores his, I can see they are just basic HTML
Are they in the .gitignore @rickydazla ?
I assume
Rick Davies
@rickydazla
Mar 19 2015 21:38
the textmate shopify bundle allows for a pages directory and can up/download
Thomas Reggi
@reggi
Mar 19 2015 21:39
to where @rickydazla
?
pages in shopify?
Rick Davies
@rickydazla
Mar 19 2015 21:39
yes
Thomas Reggi
@reggi
Mar 19 2015 21:39
it takes the html and puts it as page.content?
Rick Davies
@rickydazla
Mar 19 2015 21:39
yes
Thomas Reggi
@reggi
Mar 19 2015 21:40
hmmm
pretty interesting
Bryan Morris
@internalfx
Mar 19 2015 21:40
yeah, quickshot should get that
Rick Davies
@rickydazla
Mar 19 2015 21:40
It’s great, means you don’t have to use the shopify editor and all of your page html stays indented
Thomas Reggi
@reggi
Mar 19 2015 21:41
gem install terminal-notifier or brew install terminal-notifier and run quickshot watch | xargs -I output terminal-notifier -title Shopify -sound Tink -message output -group shopify
it works!
Bryan Morris
@internalfx
Mar 19 2015 21:41
Does anyone professionally use the Shopify editor....?
Rick Davies
@rickydazla
Mar 19 2015 21:41
And I can version control so my if my clients fudge it up I can fix
Thomas Reggi
@reggi
Mar 19 2015 21:41
I did
for a long time
Bryan Morris
@internalfx
Mar 19 2015 21:41
oh nice
Thomas Reggi
@reggi
Mar 19 2015 21:42
it pipes very nicely unix :)
The delete function is big, I have a whole bunch of files in my theme that aren't on git or my live project because I deleted them and they never were removed in the live theme.
The theme editor app never delete either.
Thomas Reggi
@reggi
Mar 19 2015 21:49
I just had a thought! If you ask shopify to extend your rate limit, and you can have more requests per second, then could you upload files faster?
The other day I dumped 100 images in my assets folder and it worked it was just pretty slow. (Hoping that quickshot wont crap out)
Bryan Morris
@internalfx
Mar 19 2015 21:50
It wont crap out :smile:
But if you get a bigger limit than quickshot will speed up :+1:
It reads the API limit headers from shopify as it makes requests, so if your limit is 100, It will try to upload almost 100 images in parallel!
By default it will do ~40
Its much faster than shopify_theme
Rick Davies
@rickydazla
Mar 19 2015 21:52
I can confirm speed=rapid
Bryan Morris
@internalfx
Mar 19 2015 21:54
quickshot does its best to destroy all your bandwidth :smile_cat:
Thomas Reggi
@reggi
Mar 19 2015 21:55
what does that look like?
I forgot about the api limit in the header
I just made an api wrapper that takes a param for requests per second :(
Bryan Morris
@internalfx
Mar 19 2015 21:56
its in the header section of the http requests...
looks like 1/40
Thomas Reggi
@reggi
Mar 19 2015 21:56
hmm
it's X-Shopify-Shop-Api-Call-Limit
not how many per second
weird
Bryan Morris
@internalfx
Mar 19 2015 21:57
You dont need to worry about the api call limit
quickshot handles it all for you, no configuration, no worries, just transfers at max speed
Bryan Morris
@internalfx
Mar 19 2015 22:04
not anymore!
Thomas Reggi
@reggi
Mar 19 2015 22:05
?
Bryan Morris
@internalfx
Mar 19 2015 22:06
I was talking about having to manually manager the API limit
Thomas Reggi
@reggi
Mar 19 2015 22:13
Right!
I need to check ideally when they raise your limit it goes from 1/40 to 1/80 or something
This might sound weird, but it says it's uploading and it's not actually updating the theme.
Bryan Morris
@internalfx
Mar 19 2015 22:17
um...?
Thomas Reggi
@reggi
Mar 19 2015 22:17
yeah
weird
Bryan Morris
@internalfx
Mar 19 2015 22:19
did you get an error?
Thomas Reggi
@reggi
Mar 19 2015 22:19
nope
it's just not reflecting the changes
had the wrong theme
:/
can we prevent quickshot from uploading itself on change :/
Bryan Morris
@internalfx
Mar 19 2015 22:23
Uploading itself?
did it upload quickshot.json
?
Thomas Reggi
@reggi
Mar 19 2015 22:24
well it tried
I had to re-run it anyway for the changes to take effect
so w.e.
Bryan Morris
@internalfx
Mar 19 2015 22:24
Im gonna file that
Rick Davies
@rickydazla
Mar 19 2015 23:19
I hit this again: Assertion failed: (end <= source_len), function SliceOnto, file ../src/smalloc.cc, line 280.
found this: joyent/node@a35ba2f
and this: iojs/io.js@c6fd2c5
Bryan Morris
@internalfx
Mar 19 2015 23:19
do you know what file it was trying to send?
Rick Davies
@rickydazla
Mar 19 2015 23:19
trying to download an image?
maybe
iojs/io.js#661
My googling skillz end there!
Bryan Morris
@internalfx
Mar 19 2015 23:22
taking a look
looks like you could try grabbing the latest copy of iojs
probably the reason I havent seen the issue...
It appears to have been a problem in the node/iojs codebase
but the fix was landed a month ago
try grabbing 1.6 and let me know how it goes https://iojs.org/en/index.html