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

remove non-working admin- and user-migrate apps, fix #26 #1542

Closed
wants to merge 2 commits into from

Conversation

jancborchardt
Copy link
Contributor

If it doesn’t work, we drop it. We can reintroduce it as soon as it works. For reference, this is the year-old bug report #26

An export without symmetric and working import just wrongly leads people to believe they can restore their data. And then issues like @srf21c describe happen – this is exactly the quality problem we always talk about.

Please review @tomneedham @DeepDiver1975 @karlitschek @MTRichards

And yes, I still do believe it’s very important to have a working export and import.

@jancborchardt
Copy link
Contributor Author

Maybe it’s only needed to remove it from the compile process for the zipfiles, but I’m not aware where that’s handled or where the list is stating which apps are part of the release.

@karlitschek
Copy link
Contributor

👎 sorry but this has to be discussed. We are also waiting for feedback from @tomneedham

@DeepDiver1975
Copy link
Contributor

👎 sorry but this has to be discussed. We are also waiting for feedback from @tomneedham

@karlitschek honestly? we are waiting for month on a solution to arrive.
last time in Stuttgart we gave this issue already a deadline - which is long passed.

👍 for killing broken features!

@karlitschek
Copy link
Contributor

Removing features is nothing we can do quickly in a pull request approved by two developers. This is a bigger discussion. involving strategy, product management, customer and user feedback. documentation, communication to users, migration paths and so on. A really appreciated activism but removing a core feature like that can't be done in a pull request coup.
Additionally we are waiting for feedback of he developer of this feature.
I will coordinate this.
Thanks you.

@jancborchardt
Copy link
Contributor Author

@tomneedham I take this pull request is obsolete since I remember you updated the apps? Do they work now?

@tomneedham
Copy link
Contributor

No the pull request is still open: owncloud/core#6005

@tomneedham
Copy link
Contributor

It works, but we need to add unit tests

@jancborchardt jancborchardt deleted the remove-admin-migrate branch January 17, 2014 18:46
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.

4 participants