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

Tree doesn't update after renaming during Duplicate action #3944

Closed
charlesh88 opened this issue Jun 14, 2021 · 7 comments · Fixed by #3947
Closed

Tree doesn't update after renaming during Duplicate action #3944

charlesh88 opened this issue Jun 14, 2021 · 7 comments · Fixed by #3947

Comments

@charlesh88
Copy link
Contributor

charlesh88 commented Jun 14, 2021

Reproduce

  1. Select an object and invoke Duplicate from its context menu.
  2. In the following dialog, rename the object. Click Save.
  3. Observe: the title in the object's main view is named properly, but in the tree it displays its old name.

image

Refreshing will fix the problem.

@jvigliotta
Copy link
Contributor

Testing

  • do what Charles said in the issue
  • just kidding (kinda)
  • duplicate an object on a server using couch for persistence
  • make sure the folder you're duping into is open
  • change the name of the duplicated object (as you duplicate)
  • verify the object shows up in the new folder (or same folder)
  • verify the object's name is changed to what you entered in the dialog

@kobe1104
Copy link
Contributor

kobe1104 commented Jun 22, 2021

Confirm fix on testathon 6/22/21.

@davetsay
Copy link
Contributor

davetsay commented Jun 22, 2021

seeing issue. if i start duplicate action, the click cancel to abort, console error

Uncaught (in promise) undefined

invoke
@
DuplicateAction.js:61

async function (async)



invoke
@
DuplicateAction.js:40

applicableActions.<computed>.callBack
@
ActionCollection.js:162

t
@
vue.min.js:6

e._withTask.e._withTask
@
vue.min.js:6

@nikhilmandlik
Copy link
Contributor

Verified Fixed.

@unlikelyzero
Copy link
Contributor

unlikelyzero commented Jun 22, 2021

Re-opening to cover the scenario outlined in #3944 (comment)

@jvigliotta
Copy link
Contributor

Updated Testing notes

  • cancel instead of choosing a location for duplication, verify no errors in console

@charlesh88
Copy link
Contributor Author

Verified fixed Testathon 06-25-21

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

Successfully merging a pull request may close this issue.

7 participants