Skip to content

Migration registration #30901

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
Merged
come-nc merged 3 commits into from
Feb 14, 2022
Merged

Migration registration #30901

come-nc merged 3 commits into from
Feb 14, 2022

Conversation

@Pytal
Copy link
Member

@Pytal Pytal commented Jan 29, 2022 •
edited
Loading

@Pytal Pytal added 2. developing Work in progress pending documentation This pull request needs an associated documentation update feature: user migration Related to User migration labels Jan 29, 2022
@Pytal Pytal added this to the Nextcloud 24 milestone Jan 29, 2022
@Pytal Pytal self-assigned this Jan 29, 2022
@Pytal Pytal mentioned this pull request Jan 29, 2022
Closed
1 task
@Pytal Pytal force-pushed the feat/30398/migration-registration branch from ef09619 to e1ebf55 Compare January 29, 2022 01:14
@Pytal Pytal mentioned this pull request Jan 29, 2022
Merged
3 tasks
@Pytal Pytal force-pushed the feat/30398/migration-registration branch 5 times, most recently from 63bdae0 to e4b0b3f Compare February 4, 2022 21:02
@Pytal Pytal added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Feb 4, 2022
@Pytal Pytal marked this pull request as ready for review February 4, 2022 21:04
@Pytal Pytal requested review from a team, PVince81, come-nc and juliusknorr and removed request for a team February 4, 2022 21:04
@Pytal Pytal force-pushed the feat/30398/migration-registration branch 2 times, most recently from 9859582 to 0999803 Compare February 8, 2022 04:22
come-nc
come-nc reviewed Feb 8, 2022
View reviewed changes
@Pytal Pytal force-pushed the feat/30398/migration-registration branch 2 times, most recently from 8e00a54 to 135a1e1 Compare February 10, 2022 06:22
PVince81
PVince81 approved these changes Feb 10, 2022
View reviewed changes
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.

👍

@Pytal Pytal removed the 3. to review Waiting for reviews label Feb 11, 2022
@Pytal Pytal added the 2. developing Work in progress label Feb 11, 2022
@Pytal Pytal mentioned this pull request Feb 11, 2022
Closed
Pytal added 3 commits February 11, 2022 23:34
@Pytal
Allow registration of migrators
e324436 
Signed-off-by: Christopher Ng <[email protected]>
@Pytal
Extend export destination interface
21f9e85 
Signed-off-by: Christopher Ng <[email protected]>
@Pytal
Add test
e485451 
Signed-off-by: Christopher Ng <[email protected]>
@Pytal Pytal force-pushed the feat/30398/migration-registration branch from 135a1e1 to e485451 Compare February 11, 2022 23:35
@Pytal Pytal added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Feb 11, 2022
@Pytal
Copy link
Member Author

Pytal commented Feb 11, 2022

Test added, would appreciate a re-review with the test :)

@come-nc come-nc merged commit 7b14b96 into master Feb 14, 2022
@come-nc come-nc deleted the feat/30398/migration-registration branch February 14, 2022 11:40
@timm2k timm2k changed the title Migration registraton Migration registration Feb 15, 2022
@Pytal Pytal removed the pending documentation This pull request needs an associated documentation update label Nov 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@PVince81 PVince81 PVince81 approved these changes

@artonge artonge artonge approved these changes

@come-nc come-nc come-nc approved these changes

@juliusknorr juliusknorr Awaiting requested review from juliusknorr

Assignees

@Pytal Pytal

Labels

3. to review Waiting for reviews feature: user migration Related to User migration

Projects

None yet

Milestone

Nextcloud 24

Development

Successfully merging this pull request may close these issues.

5 participants

@Pytal @PVince81 @artonge @come-nc