Skip to content
This repository was archived by the owner on Jan 6, 2023. It is now read-only.

Conversation

@icyflame
Copy link
Contributor

@icyflame icyflame commented Oct 7, 2016

If a non-standard documentRoot is used, then upload doesn't work. upload always tries to upload www.zip, although Utils.createArchive is creating the archive with the same name as in documentRoot inside the project config.

Fixes #121

@icyflame
Copy link
Contributor Author

icyflame commented Oct 7, 2016

I am unable to understand why this test is failing.

It seems that the error is being thrown from the only place where the function split has been called which is in lib/project.js.

I have tested this on my machine with an actual project using npm link and it is working fine. (Tested against this branch )

- Ensures that the package.json's default directory for the code is considered
  before suggesting the zip path

Signed-off-by: Siddharth Kannan <kannan.siddharth12@gmail.com>
@icyflame icyflame force-pushed the fix-zip-path-before-upload branch from 6dad276 to 263a0b5 Compare October 18, 2016 19:02
@icyflame
Copy link
Contributor Author

ping @jthoms1 and @tlancina ! Thanks a lot for merging #124. I have rebased this branch to the latest master. Please have a look! 🙂

@icyflame
Copy link
Contributor Author

icyflame commented Nov 1, 2016

ping @jthoms1 and @tlancina.

1 similar comment
@icyflame
Copy link
Contributor Author

ping @jthoms1 and @tlancina.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant