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

Is remove from changelist feature removed? #569

Closed
MichaelSuen-thePointer opened this issue May 10, 2019 · 11 comments
Closed

Is remove from changelist feature removed? #569

MichaelSuen-thePointer opened this issue May 10, 2019 · 11 comments

Comments

@MichaelSuen-thePointer
Copy link

Cannot find remove from changelist feature in the right click menu anymore.

Is it removed or moved to other place?

If it is removed, please change this issue to a feature request, I will be grateful.

image
image

@edgardmessias
Copy link
Contributor

If file has a changelist, on "Set changelist" has option to remove.

Look:

if (canRemove) {

@MichaelSuen-thePointer
Copy link
Author

If file has a changelist, on "Set changelist" has option to remove.

Look:

if (canRemove) {

Tried, but there's no remove option.
test

@edgardmessias
Copy link
Contributor

@JohnstonCode, I can't reproduce, working fine here.
@MichaelSuen-thePointer , what you extension version? VSCode?
Go to "Help > Toggle Developers Tools" and check some message

@MichaelSuen-thePointer
Copy link
Author

MichaelSuen-thePointer commented May 13, 2019

@JohnstonCode, I can't reproduce, working fine here.
@MichaelSuen-thePointer , what you extension version? VSCode?
Go to "Help > Toggle Developers Tools" and check some message

VSCode:
version: 1.33.1 (user setup)
commit: 51b0b28134d51361cf996d2f0a1c698247aeabd8
date: 2019-04-11T08:27:14.102Z
Electron: 3.1.6
Chrome: 66.0.3359.181
Node.js: 10.2.0
V8: 6.6.346.32
OS: Windows_NT x64 10.0.16299

Extension: 1.48.2

After a quick debugging, I found this repository.changelists is just an empty object, wierd.
Where is it initialized?

image

BTW, every time I left click on the file which has been added to the changelist (like the one below)
image
an 'oh no' error message appears in the debug console, don't know why.

@JohnstonCode
Copy link
Owner

@edgardmessias
Copy link
Contributor

@MichaelSuen-thePointer, the question is: How the file is in "IGNORE-ON-COMMIT"? You can set another change list and back?

@JohnstonCode
Copy link
Owner

@edgardmessias I cant reproduce either

@MichaelSuen-thePointer
Copy link
Author

@edgardmessias I cant reproduce either

@JohnstonCode, I can't reproduce, working fine here.
@MichaelSuen-thePointer , what you extension version? VSCode?
Go to "Help > Toggle Developers Tools" and check some message

Guys, I found a way to repro the bug:

  1. checkout a folder, lets say folder1
  2. modify folder1's externals property: add an external entry to another folder, lets say folder2
    image
  3. commit changes in folder1 and update in folder1 to get folder2 downloaded
  4. create a file testfile.txt inside folder1/folder2
  5. commit testfile.txt
    image
  6. set testfile.txt's change list to ignore-on-commit, either in VSCode or in command line is OK
  7. back to folder1, open with Code here
  8. see what happends
    image

@MichaelSuen-thePointer
Copy link
Author

@JohnstonCode @edgardmessias
Guys, I found the steps to reproduce the issue above, please have a try.

@JohnstonCode
Copy link
Owner

Thanks for finding a way to replicate this. Will have a look as soon as i have time.

edgardmessias added a commit to edgardmessias/svn-scm that referenced this issue Jun 3, 2019
JohnstonCode added a commit that referenced this issue Jun 3, 2019
## [1.50.4](v1.50.3...v1.50.4) (2019-06-03)

### Bug Fixes

* Fixed changelists for externals (close [#569](#569)) ([#597](#597)) ([3aa381f](3aa381f))
@JohnstonCode
Copy link
Owner

🎉 This issue has been resolved in version 1.50.4 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants