Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Error on watching multiple sub-directories #70

Closed
valterlima opened this issue Nov 30, 2017 · 6 comments
Closed

Error on watching multiple sub-directories #70

valterlima opened this issue Nov 30, 2017 · 6 comments

Comments

@valterlima
Copy link

Hello,

Thanks and congratulations for the plugin, it's a really great tool :)

I'm having a issue when I have a dw.json like this:

"cartridge": [
"xxxxxxxx/abcd",
"sitegenesis/app_storefront_core",
"sitegenesis/app_storefront_controllers"
]

Every time I change some code from either directory, Prophet tries to do a re-upload everything. Example:
[U 10:33:27] c:\projects\xxxxxxxx\abcd\file.js
Trying to re-upload
Start uploading cartridges

I know this isn't the correct configuration for the paths as indicated in the plugin description, but it's
very useful when we have multiple GIT repositories

I'm using the latest version from the Plugin and VS Code in Windows 10.

Best regards,

@SqrTT
Copy link
Owner

SqrTT commented Nov 30, 2017

Hi.
Isn't this issue a duplicate of #55?

@valterlima
Copy link
Author

valterlima commented Nov 30, 2017

They are similar, but I detected that the error occurs when we have one more nested directory. With dwupload we have a similar issue as well (an internal error happens).

@SqrTT
Copy link
Owner

SqrTT commented Dec 4, 2017

ahh, I see. Workaround for this issue could be a usage of symlinks ( #62 ).

The correct way would be to use multiroot workspaces but currently, the extension doesn't support it.

@CarolMinichOSF
Copy link

CarolMinichOSF commented Dec 11, 2017

I confirm what Valter said . Will the extension support multiroot workspaces in the near future ? we sometimes have to make 50+ symlinks . Thank you

@SqrTT
Copy link
Owner

SqrTT commented Dec 11, 2017

Yes, there are plans to add multiroot workspaces and SDAPI v2. Currently, I don't have a much time for the extension. I'm planning to do so in the first quarter of 2018.

@SqrTT
Copy link
Owner

SqrTT commented Mar 20, 2018

the multiroot has been added in v1.0.0

@SqrTT SqrTT closed this as completed Mar 22, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants