ember-cli-deploy-original icon indicating copy to clipboard operation
ember-cli-deploy-original copied to clipboard

deploy:index but forgot to build first

Open jbrown opened this issue 11 years ago • 10 comments

Then you're stuck. :flushed:

Version for key [e105df69ae] has already been uploaded

jbrown avatar Nov 25 '14 18:11 jbrown

:+1: having a similar issue where even when my files under dist are changing, it keeps wanting to use the same version key and fails with the same error

zpatten avatar Nov 25 '14 23:11 zpatten

$ npm list | grep ember-cli-deploy
npm├─┬ [email protected]
│ ├─┬ [email protected]
 ├─┬ [email protected]

zpatten avatar Nov 25 '14 23:11 zpatten

The version key used is the current git hash. Therefore, if you've made changes, you need to commit them, then build, then deploy. If you make changes, build, deploy then you are still using the same git hash.

achambers avatar Nov 26 '14 09:11 achambers

I did exactly that and still had the same results; it appeared to get stuck on a specific key.

zpatten avatar Dec 17 '14 21:12 zpatten

Hey @jbrown, this isn't due to not having built first. It's because you haven't committed. Because the current index adapter we use the git hash as the key.

How would you expect this to work?

Also, this may not be an ember-cli-deploy change but more so an ember-cli-redis-index-adapter change

achambers avatar Dec 24 '14 07:12 achambers

@achambers Unless you have a post commit hook that builds your production bundle, it's pretty easy to forget to build between committing and deploying. In previous versions you could just overwrite a key but that functionality is gone now which leaves no room for error. How do you feel about a --force flag that allows you to overwrite a key?

jbrown avatar Dec 30 '14 09:12 jbrown

Rather than using the git hash, just hash the file? Then you'd have a unique hash if the file had changed. Isn't this how fingerprinting works?

BryanCrotaz avatar Jan 04 '15 09:01 BryanCrotaz

ember-cli-deploy is in the process of merging with ember-deploy. Therefore, no more changes will be made to this repo. Please see the README for more details. /cc @jbrown @zpatten

achambers avatar Feb 27 '15 09:02 achambers

@achambers Thanks for the update, but considering your response is around 3 months old I've long since moved on.

zpatten avatar Mar 02 '15 20:03 zpatten

No problems @zpatten. These changes won't effect you then.

achambers avatar Mar 02 '15 20:03 achambers