Skip to content

Fixed: livesync + debug syncs wrong files on Android #1994

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

Merged
merged 1 commit into from
Aug 11, 2016

Conversation

tzraikov
Copy link
Contributor

@tzraikov tzraikov commented Aug 10, 2016

  1. Create a new {N} project.
  2. Run tns livesync android --watch.
  3. Do some changes in XML and JS files.
  4. Stop tns process.
  5. Open Visual Studio Code and start Sync on Android action.
  6. Change JS file - you will see that the JS file is correct, but the XML one is not.
  7. Change XML file - you will see that the XML file is correct, but the JS file is not.

Should be merged on top of #1985

@lukashlobil
Copy link

Hi, do you think this has some relation to my issue on stack overflow ? Basically tns livesync builds old code (regardless of IDE), but it builds even wrong tj (js).
http://stackoverflow.com/questions/38853359/nativescript-builds-old-code

@tzraikov tzraikov force-pushed the raikov/fix-livesync-debug-android-wrong-files branch from ae9986a to a92cccd Compare August 11, 2016 13:12
@tzraikov
Copy link
Contributor Author

tzraikov commented Aug 11, 2016

@kekelzpekel: Yes, this could be the same issue. It could not make it to our latest release - 2.2.0. It will be included in 2.2.1 that will be available next week.

@tzraikov tzraikov merged commit 41d18e9 into master Aug 11, 2016
@tzraikov tzraikov deleted the raikov/fix-livesync-debug-android-wrong-files branch August 11, 2016 13:14
tzraikov added a commit that referenced this pull request Aug 11, 2016
…android-wrong-files

Fixed: livesync + debug syncs wrong files on Android
@tzraikov tzraikov added this to the 2.2.1 milestone Aug 12, 2016
@vchimev vchimev self-assigned this Aug 12, 2016
@vchimev vchimev removed their assignment Jun 8, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants