Skip to content
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

Editor file content synchronization and git checkout notification #2153

Closed
wants to merge 16 commits into from
Closed

Editor file content synchronization and git checkout notification #2153

wants to merge 16 commits into from

Conversation

dkuleshov
Copy link

What does this PR do?

  • Adds means to synchronize content of file opened in editor when it is being modified outside of che
  • Adds means to track outside git check out operations (branch checkout or revision checkout) and inform a user

What issues does this PR fix or reference?

#1910
#1912

New Behavior

If file that is opened in che editor tab is modified somewhere else its content is updated
If git repository is checked out outside of che UI a user is notified about that

Tests written?

Yes

Dmitry Kuleshov added 2 commits August 17, 2016 12:59
Signed-off-by: Dmitry Kuleshov <dkuleshov@codenvy.com>
Signed-off-by: Dmitry Kuleshov <dkuleshov@codenvy.com>
@codenvy-ci
Copy link

Build # 117 - FAILED

Please check console output at https://ci.codenvycorp.com/job/che-pullrequests-build/117/ to view the results.

@dkuleshov
Copy link
Author

ci-build

@vparfonov
Copy link
Contributor

ok

@codenvy-ci
Copy link

Build # 122 - FAILED

Please check console output at https://ci.codenvycorp.com/job/che-pullrequests-build/122/ to view the results.

@codenvy-ci
Copy link

Build # 141 - FAILED

Please check console output at https://ci.codenvycorp.com/job/che-pullrequests-build/141/ to view the results.

@codenvy-ci
Copy link

Build # 144 - FAILED

Please check console output at https://ci.codenvycorp.com/job/che-pullrequests-build/144/ to view the results.

@codenvy-ci
Copy link

Build # 146 - FAILED

Please check console output at https://ci.codenvycorp.com/job/che-pullrequests-build/146/ to view the results.

@codenvy-ci
Copy link

Build # 147 - FAILED

Please check console output at https://ci.codenvycorp.com/job/che-pullrequests-build/147/ to view the results.

@codenvy-ci
Copy link

Build # 148 - FAILED

Please check console output at https://ci.codenvycorp.com/job/che-pullrequests-build/148/ to view the results.

@codenvy-ci
Copy link

Build # 149 - FAILED

Please check console output at https://ci.codenvycorp.com/job/che-pullrequests-build/149/ to view the results.

@codenvy-ci
Copy link

Build # 150 - FAILED

Please check console output at https://ci.codenvycorp.com/job/che-pullrequests-build/150/ to view the results.

@codenvy-ci
Copy link

@codenvy-ci
Copy link

@codenvy-ci
Copy link

@codenvy-ci
Copy link

@vparfonov vparfonov closed this Sep 14, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants