Skip to content

Conversation

@CarlSchwan
Copy link
Member

@CarlSchwan CarlSchwan commented Feb 1, 2022

This makes it a bit easier to transfer ownership when the new user
hasn't already logged in. This still doesn't support encrypted storage because the keys are not generated yet (this is already checked at the beginning of the method).

@CarlSchwan CarlSchwan added the 3. to review Waiting for reviews label Feb 1, 2022
@CarlSchwan CarlSchwan added this to the Nextcloud 24 milestone Feb 1, 2022
@CarlSchwan CarlSchwan force-pushed the feat/init-storage-when-transferring-ownership branch 2 times, most recently from 154067e to 352c4be Compare February 1, 2022 09:58
@PVince81
Copy link
Member

PVince81 commented Feb 1, 2022

with "still doesn't support storage" do you mean encrypted storage ?

@CarlSchwan
Copy link
Member Author

with "still doesn't support storage" do you mean encrypted storage ?

yes

Copy link
Member

@PVince81 PVince81 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 as discussed

This makes it a bit easier to transfer ownership when the new user
hasn't already logged in. This still doesn't support encrypted
storages because the keys are not generated yet.

Signed-off-by: Carl Schwan <[email protected]>
@CarlSchwan CarlSchwan force-pushed the feat/init-storage-when-transferring-ownership branch from 352c4be to 02da4ce Compare February 1, 2022 12:26
@PVince81
Copy link
Member

/backport to stable23

@PVince81
Copy link
Member

/backport to stable22

@PVince81
Copy link
Member

/backport to stable21

@PVince81 PVince81 merged commit 62d4710 into master Feb 10, 2022
@PVince81 PVince81 deleted the feat/init-storage-when-transferring-ownership branch February 10, 2022 17:13
@PVince81
Copy link
Member

PVince81 commented Mar 4, 2022

/backport to stable23

@PVince81
Copy link
Member

PVince81 commented Mar 4, 2022

/backport to stable22

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

Labels

3. to review Waiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants