Skip to content
This repository has been archived by the owner on Nov 14, 2018. It is now read-only.

Update admin_migrate for ownCloud 6. #1546

Closed
wants to merge 2 commits into from
Closed

Conversation

tomneedham
Copy link
Contributor

Change UI to show just one button for exporting the config file.

This pull request is complete, but must wait for another request to be completed in the core repo.

@@ -1,12 +1,9 @@
<form id="export" action="#" method="post">
<fieldset class="personalblock">
<h2><?php p($l->t('Export data'));?></h2>
<h2><?php p($l->t('Admin Export'));?></h2>
Copy link
Contributor

Choose a reason for hiding this comment

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

The title should be »Config export« to be the same as in the app list. :)

@jancborchardt
Copy link
Contributor

@tomneedham what’s the status on this?

@tomneedham
Copy link
Contributor Author

@jancborchardt We need a decision on what we are going to finally do with both of these apps, and when we make the change.

Following our discussion with @DeepDiver1975 in Stuttgart, I think we agreed the user_migrate app is fairly redundant. Most apps provide methods to export data individually, and file data is usually too large, but can already be zipped and streamed to the user in the files app.

Admin_migrate is affected by the majority of problems mentioned above. It's only real use case is to allow admins with owncloud instances on shared hosting to download a copy of the config file if they don't have access.

TL;DR IMO we depreciate user_migrate, admin_migrate and instead create a new config_export that exports the admin config. Possibly in the future we could create an app to export the database for users of shared hosting.

@jancborchardt
Copy link
Contributor

@tomneedham good plan, I think I agree on that. @DeepDiver1975 what do you think?

@tomneedham
Copy link
Contributor Author

@DeepDiver1975 Ping. We should fix this before OC7 right?

@DeepDiver1975
Copy link
Contributor

TL;DR IMO we depreciate user_migrate, admin_migrate and instead create a new config_export that exports the admin config. Possibly in the future we could create an app to export the database for users of shared hosting.

yes - that's the way to go from my point of view - what we should think about:
this feature is broken now for month and actually nobody was crying for it: do we really need that aka the time might be better invested into other topics

@tomneedham
Copy link
Contributor Author

Thanks. I'll put together a pull request.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants