add --destination-use-user-folder option to files:transfer-ownership command#3993
Conversation
|
@mrow4a you need to add the new option also to the corresponding occ command set file |
mrow4a
force-pushed
the
tr-own-destination-use-user-folder-opt
branch
2 times, most recently
from
a9bd095 to
e4fcbaf
Compare
|
@mmattel done, but not sure if correctly |
modules/admin_manual/pages/configuration/server/occ_commands/core_commands/file_commands.adoc
Show resolved
Hide resolved
modules/admin_manual/pages/configuration/server/occ_commands/core_commands/file_commands.adoc
Outdated
Show resolved
Hide resolved
modules/admin_manual/pages/configuration/server/occ_commands/core_commands/file_commands.adoc
Outdated
Show resolved
Hide resolved
phil-davis
force-pushed
the
tr-own-destination-use-user-folder-opt
branch
from
248a490 to
1951342
Compare
|
Squashed and rebased. |
|
This is a new thing that will be in the next oC10.9 release. So no backport needed. |
| --accept-skipped-shares` | always confirm to continue in case of skipped shares. | ||
| | `--destination-use-user-folder` | if destination user never logged in + | ||
| and thus has no mounts transfer + | ||
| directly to top-level user folder. |
There was a problem hiding this comment.
The first line is a if
followed by a statemet - ok
but the next line does not point to a solution or explanation (directly to top-level user folder).
The sentence looks "strange/confusing" Please improve.
Well done, your changes render the following ✔️
There was a problem hiding this comment.
any suggestions how to improve? my docs were really engineer driven
There was a problem hiding this comment.
Yes, this is difficult to parse. A comma after "mounts" should clarify things. Then the share is transferred directly to the top-level user folder. Is that what you meant? @mrow4a
There was a problem hiding this comment.
@mmattel @EParzefall adjusted, can you check now ?
|
@mrow4a thansk for the update. Q: Does this mean the following: If the target user never logged in, there is no skeleton on a directory level present for this user. What happens if this option is not set and the user never logged in? |
|
@mmattel updated |
|
@mrow4a thanks for updating - approved. No backport needed, will go into 10.9 |
delete-merged-branch
bot
deleted the
tr-own-destination-use-user-folder-opt
branch
4 participants
New option for
occ files:transfer-ownershipPerform transfer of files and shares from /files/source-user-id to /files/dest-user-id
References: owncloud/core#39118