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.

Sign up for GitHub

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

Jump to bottom

Add move rename support #43

Merged
merged 2 commits into from
Oct 5, 2016
Merged

Add move rename support #43

merged 2 commits into from
Oct 5, 2016

Conversation

nickvergessen
Copy link
Member

@nickvergessen nickvergessen commented Sep 15, 2016 •
edited
Loading

Steps

  1. Create test/welcome.txt
  2. Create test/sub/
  3. Create test/sub2/
  4. Share test/ with user test
  5. Share test/sub/ with user foo
  6. Share test/sub2/ with user bar
  7. Clear activity table
  8. Run php moveCross.php (DONT WEBDAV MOVE, its a create+delete activity (not recoverable))

Expected

  • test has a move activity
  • foo has a delete activity
  • bar has a create activity

@rullzer want to test? 😺

@nickvergessen nickvergessen added this to the Nextcloud 11.0 milestone Sep 15, 2016
@nickvergessen nickvergessen mentioned this pull request Sep 26, 2016
Merged
1 task
@MorrisJobke
Copy link
Member

Tested and works 👍

@MorrisJobke MorrisJobke mentioned this pull request Oct 5, 2016
Closed
47 tasks
@rullzer
Copy link
Member

rullzer commented Oct 5, 2016

LGTM

@nickvergessen nickvergessen merged commit 8891ade into master Oct 5, 2016
@nickvergessen nickvergessen deleted the add-move-rename-support branch October 5, 2016 08:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
3. to review enhancement feature: implementations
Projects
None yet
Milestone
Nextcloud 11.0
Development

Successfully merging this pull request may close these issues.
3 participants
@nickvergessen @MorrisJobke @rullzer