Skip to content
This repository has been archived by the owner on Oct 1, 2018. It is now read-only.

deploy via ftp #4

Closed
pablomaurer opened this issue Sep 17, 2015 · 7 comments
Closed

deploy via ftp #4

pablomaurer opened this issue Sep 17, 2015 · 7 comments
Milestone

Comments

@pablomaurer
Copy link

I don't have amazon stuff.. =) It's no problem to deploy it to my server, but doing this via the cli would be very nice. So maybe add some FTP or other options for deploying.

Ups just saw More deployment targets will be added later, but an issue is anyway ok, so people can track stuff.

PS: very nice stuff you made..!! Will test it deeply.

@nikDemyankov
Copy link
Member

Thanks for the feedback! Yes, we absolutely need to extend deployment mechanism. Will post update information here as soon as I'll have any. But you always welcomed to contribute :)

@pablomaurer
Copy link
Author

sorry, but maybe it's deplyoment related, so i aks it here. I'm getting the error -4.
I made an extra easy manifest, to test:

[
  {
    "file": "C:\\projekte\\ai\\project\\www\\css\\style.css",
    "hash": "0699289c56ae49c509e03e49fdbd0b1b"
  }
]
  1. I deployed the app to IOS with following files from you:
TestProject/
  www/
    css/styles.css
    chcp.json         // old
    chcp.manifest  // old
  cordova.hcp.json
  1. I changed style.css
  2. I run cordova-hcp build
  3. It changed -> chcp.json & chcp.manifesft
  4. I deployed the update to my server, like that:
www/
  css/styles.css // here
chcp.json  // new
chcp.manifest  // new
styles.css // or here, tried both

so what am I doing wrong?
ok, got it -> I have to take the chcp.manifest on the server and change these auto generated absolute pathes to relative pathes

so what am I doing wrong, this time?
now it logs, that it's updating and the app is getting reloaded, but the changes i made are not appearing.

@nikDemyankov
Copy link
Member

Not sure this is a CLI issue, more like a question on a plugin.
Manifest file is incorrect:

[
  {
    "file": "C:\\projekte\\ai\\project\\www\\css\\style.css",
    "hash": "0699289c56ae49c509e03e49fdbd0b1b"
  }
]

file should be a path, that is relative to www folder. Plugin will append path from the file to the content_url you specified in chcp.json file.

More details on manifest file can be found here.

One more note on deployment: you should deploy only stuff from the www folder, but www folder itself should not be there. For example, let say you have a server on http://somecoolserver.com, your cordova project has structure:

TestProject/
  config.xml
  hooks/
  node_modules/
  platforms/
  plugins/
  www/
    chcp.manifest
    index.html
    css/
      styles.css

You should deploy your www folder content so it would be accessible like that:

http://somecoolserver.com/index.html
http://somecoolserver.com/css/style.css
http://somecoolserver.com/chcp.manifest

@nikDemyankov
Copy link
Member

On the other hand, if cordova-hcp build generated file path as you showed (C:\\projekte\\ai\\project\\www\\css\\style.css) - then yes, it is a bug of CLI client on Windows machine. We will investigate that.

@pablomaurer
Copy link
Author

Yes not really an issue..
But why does then the build command generate:
"file": C:\\projekte\\ai\\project\\www\\css\\style.css" instead of
"file": "style.css"

Ok you got it.. Started to write this a while ago. writing this comment from mobile device is hard..

I will create a full ionic example in order to help others and to try recreate the bug that it reloads the app but, doesnt show the updated css changes..

But anyway thanks alot and cool that your helping me ;)
Maybe i also can help on some other stuff, but I dont have the time now, to learn objective-c.

@nikDemyankov
Copy link
Member

No, thank you for the feedback! It is really cool to know that people testing our plugin.

CLI client is fully JavaScript, so you don't have to learn any mobile-native languages, if you want to contribute :) Contribution is not only coding something, but also testing and providing ideas on how to improve. So yeah, you already do that :)

Created new issue regarding windows and build command: issue #5. All further discussion should be continued in there.

@nikDemyankov nikDemyankov added this to the v1.1.0 milestone Dec 4, 2015
@nikDemyankov nikDemyankov modified the milestones: v1.2.0, v1.1.0 May 4, 2016
@nordnet-deprecation-bot
Copy link
Contributor

👋 Hi! Thank you for your interest in this repo.

😢 We are not using nordnet/cordova-hot-code-push-cli anymore, and we lack the manpower and the experience needed to maintain it. We are aware of the inconveniece that this may cause you. Feel free to use it as is, or create your own fork.

🔒 This will now be closed & locked.

ℹ️ Please see #79 for more information.

@nordnet nordnet locked and limited conversation to collaborators Sep 19, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants