Skip to content

Import guest user data into system user account at first login#208

Merged
rullzer merged 2 commits intomasterfrom
feature/migrate-first-user-login
Dec 2, 2019
Merged

Import guest user data into system user account at first login#208
rullzer merged 2 commits intomasterfrom
feature/migrate-first-user-login

Conversation

@ChristophWurst
Copy link
Member

@ChristophWurst ChristophWurst commented Nov 19, 2019 •
edited
Loading

@ChristophWurst ChristophWurst added the 2. developing Work in progress label Nov 19, 2019
@ChristophWurst ChristophWurst self-assigned this Nov 19, 2019
@ChristophWurst ChristophWurst mentioned this pull request Nov 20, 2019
Merged
@ChristophWurst ChristophWurst changed the title Trigger a user data transfer once the guest user got a real account Import guest user data into system user account at first login Nov 21, 2019
@ChristophWurst ChristophWurst force-pushed the feature/migrate-first-user-login branch from 0d8b8c3 to 10806c2 Compare November 26, 2019 08:40
@ChristophWurst ChristophWurst marked this pull request as ready for review November 26, 2019 08:41
@ChristophWurst ChristophWurst added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Nov 26, 2019
@rullzer rullzer force-pushed the feature/migrate-first-user-login branch from 10806c2 to c36ecbc Compare November 28, 2019 08:57
@rullzer
Copy link
Member

rullzer commented Nov 28, 2019

Pushed some fixes from my POV

Seems we do not transfer shares with the transfer ownership. Which is the main thing required here since guests have no quota by default.

ANyways added that and seems to work.

@rullzer rullzer force-pushed the feature/migrate-first-user-login branch from 860bf39 to 89b2288 Compare December 2, 2019 12:40
@ChristophWurst ChristophWurst added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Dec 2, 2019
@ChristophWurst @rullzer
Trigger a user data transfer once the guest user got a real account
9429219 
Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
@rullzer rullzer force-pushed the feature/migrate-first-user-login branch from 7c1247f to dd6d695 Compare December 2, 2019 19:17
@ChristophWurst @rullzer
Fix tests against latest Nextcloud master
ebd5cdc 
Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
@rullzer rullzer force-pushed the feature/migrate-first-user-login branch from dd6d695 to ebd5cdc Compare December 2, 2019 19:22
@rullzer rullzer merged commit 3b1132b into master Dec 2, 2019
@rullzer rullzer deleted the feature/migrate-first-user-login branch December 2, 2019 20:05
@wiswedel
Copy link
Contributor

wiswedel commented Dec 13, 2019

resolves #105

This was referenced Dec 13, 2019
Closed
Closed
@nextcloud nextcloud deleted a comment from wiswedel Dec 13, 2019
@schiessle schiessle mentioned this pull request Jun 18, 2020
Closed
@PVince81 PVince81 added the pending documentation Pending a documentation update/fix label Aug 9, 2021
@PVince81 PVince81 mentioned this pull request Aug 9, 2021
Closed
@PVince81 PVince81 removed the pending documentation Pending a documentation update/fix label Aug 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rullzer rullzer rullzer approved these changes

Assignees

@ChristophWurst ChristophWurst

Labels

4. to release Ready to be released and/or waiting for tests to finish

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@ChristophWurst @rullzer @wiswedel @PVince81