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

fix when app verlay installation,www content update filed#225

Merged
nikDemyankov merged 2 commits into
nordnet:masterfrom
zwwhappy:master
Sep 27, 2016
Merged

fix when app verlay installation,www content update filed#225
nikDemyankov merged 2 commits into
nordnet:masterfrom
zwwhappy:master

Conversation

@zwwhappy

Copy link
Copy Markdown
Contributor

currentRelease is equals newRelease so get error is code: -7, description: "Can't copy files from previous release to the new release"

fix when app verlay installation,www content update filed
error is code: -7, description: "Can't copy files from previous release to the new release"
fix when app verlay installation,www content update filed
error is code: -7, description: "Can't copy files from previous release to the new release"
@nikDemyankov

Copy link
Copy Markdown
Member

Thank you for the pull request! Did I understand that correctly: error occurs when plugin tries to install release, that it's already is using? Maybe better would be just stop the whole installation process?

@zwwhappy

Copy link
Copy Markdown
Contributor Author

It's already is using. I think is better than stop installation process.When server has no new release version, the installation process and update from server is run normally.

@nikDemyankov

Copy link
Copy Markdown
Member

But if the app is already using a newest version - plugin should not re-install it, since there is no point in that... Or am I missing something?)

This was referenced Sep 26, 2016
@nikDemyankov

Copy link
Copy Markdown
Member

I think I found what might be the causing that. Will merge this PR and add some more changes to it. Thanks!

@nikDemyankov nikDemyankov merged commit ba3276f into nordnet:master Sep 27, 2016
@nikDemyankov nikDemyankov added this to the v1.5.2 milestone Sep 27, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants