Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • Jul 25 17:53
    rickydazla commented #61
  • Jul 25 17:52
    internalfx commented #61
  • Jul 23 21:20
    rickydazla edited #61
  • Jul 23 21:20
    rickydazla edited #61
  • Jul 23 21:17
    rickydazla commented #61
  • Jul 23 14:34
    valespring commented #65
  • Jul 23 13:05
    internalfx commented #65
  • Jul 22 04:39
    valespring commented #65
  • Jul 22 04:31
    valespring edited #65
  • Jul 22 04:24
    valespring edited #65
  • Jul 22 04:23
    valespring opened #65
  • May 03 18:34
    valespring commented #55
  • May 02 22:19
    internalfx closed #64
  • May 02 22:19
    internalfx commented #64
  • May 02 22:16
    valespring commented #64
  • May 02 16:06
    internalfx opened #64
  • May 02 16:01
    internalfx opened #63
  • May 02 16:00
    internalfx closed #62
  • May 02 16:00
    internalfx commented #62
  • May 02 15:55
    valespring commented #55
Rick Davies
@rickydazla
ok
04:28:14 pm - Error: EEXIST: file already exists, mkdir '/Users/Rick/Sites/myshopify/th
emes/taylorstitch/theme/assets'
=== START OF OUTPUT ===
{ Error: EEXIST: file already exists, mkdir '/Users/Rick/Sites/myshopify/themes/taylors

titch/theme/assets'
  cause:
   { Error: EEXIST: file already exists, mkdir '/Users/Rick/Sites/myshopify/themes/tayl
orstitch/theme/assets'
     errno: -17,
     code: 'EEXIST',
     syscall: 'mkdir',
     path: '/Users/Rick/Sites/myshopify/themes/taylorstitch/theme/assets' },
  isOperational: true,
  errno: -17,
  code: 'EEXIST',
  syscall: 'mkdir',
  path: '/Users/Rick/Sites/myshopify/themes/taylorstitch/theme/assets' }
==== END OF OUTPUT ====
Bryan Morris
@internalfx
hmm....I wonder if that is MAC specific?
I basically tell the OS to make the folder if it doesn't exist....
Rick Davies
@rickydazla
it does already exist
i’m (re)downloading an existing theme
to check if there are updates on the live site that were made by administrators and 3rd party apps
… which is why I wonder if I should be running qs theme watch --sync instead
Bryan Morris
@internalfx
no...download is what you want...
That is why I'm curious if this is something MAC does differently....my machine seems to silently ignore mkdir if the folder exists.
So it produces no error and writes the file.
Rick Davies
@rickydazla
oh oh i see
¯_(ツ)_/¯
Bryan Morris
@internalfx
give 3.0.7 a whirl
I wrapped the mkdir logic in a try lets see if it works now...
It may just be something I need to do to properly support MAC
Rick Davies
@rickydazla
whoah
04:39:34 pm - Downloaded templates/gift_card.liquid
{ Error: EEXIST: file already exists, mkdir '/Users/Rick/Sites/myshopify/themes/taylors
titch/theme/templates/customers'

  cause:
   { Error: EEXIST: file already exists, mkdir '/Users/Rick/Sites/myshopify/themes/tayl
orstitch/theme/templates/customers'
     errno: -17,
     code: 'EEXIST',
     syscall: 'mkdir',
     path: '/Users/Rick/Sites/myshopify/themes/taylorstitch/theme/templates/customers'
},
  isOperational: true,
  errno: -17,
  code: 'EEXIST',
  syscall: 'mkdir',
  path: '/Users/Rick/Sites/myshopify/themes/taylorstitch/theme/templates/customers' } '
the folder is already there'
04:39:34 pm - Downloaded templates/customers/register.liquid
{ Error: EEXIST: file already exists, mkdir '/Users/Rick/Sites/myshopify/themes/taylors
titch/theme/templates/customers'
  cause:
   { Error: EEXIST: file already exists, mkdir '/Users/Rick/Sites/myshopify/themes/tayl
orstitch/theme/templates/customers'
     errno: -17,
     code: 'EEXIST',
     syscall: 'mkdir',
     path: '/Users/Rick/Sites/myshopify/themes/taylorstitch/theme/templates/customers'
},
  isOperational: true,
  errno: -17,
  code: 'EEXIST',
  syscall: 'mkdir',
  path: '/Users/Rick/Sites/myshopify/themes/taylorstitch/theme/templates/customers' } '
the folder is already there'
04:39:34 pm - Downloaded templates/customers/reset_password.liquid
file error file error file error
Bryan Morris
@internalfx
Did it actually get the files?
Rick Davies
@rickydazla
yes
Bryan Morris
@internalfx
nice...I went ahead and kept logging the error so we could see it.
BOOM 3.0.8
I got a good feeling about this one.
it hides EEXIST errors when trying to create folders. We don't care if they already exists.
Rick Davies
@rickydazla
:boom:
thanks
Bryan Morris
@internalfx
Why linux is naturally OK with that and MAC isn't? i dunno
Appreciated the heads up and testing....
Not supporting MAC is like not supporting probably 90% of my userbase :laughing:
Rick Davies
@rickydazla
haha
Bryan Morris
@internalfx
Right, so is your config still messed up?
Rick Davies
@rickydazla
No it’s fine… I figured out what changed and did search and replace. It did reject the old config, but I had like 16 targets and I know I’m not alone in that, so my question was really if there could be an automagic way (or even docs as to what changed)?
Bryan Morris
@internalfx
right...I basically turned 3 settings into one.
To make creating configs much less tedious
Rick Davies
@rickydazla
yeah, i like it
Bryan Morris
@internalfx
Now all you need is a name and the "example URL" given from the private app page.
I realized all the info I needed was in that URL so I parsed the data out of it.
I bet you noticed quickshot3 now pretty formats the quickshot.json :smile:
no super long string of JSON
Rick Davies
@rickydazla
yes, that’s great actually
b/c i usually create new targets by editing that directly but i had to beautify it each time
Bryan Morris
@internalfx
@rickydazla there is now a breaking changes section....
Rick Davies
@rickydazla
now we’re cookin on gas!
Trajce
@trajce
I just checked this thread, so qs v3 is official :D
Ill give it a try on windows 10 and ping you back if i got any specific notes
Bryan Morris
@internalfx
@trajce Please Do! I work and develop on Linux so it's a big help when people let me know problems they find on other platforms
Trajce
@trajce
It seems like its working fine, no issues so far on windows.