search

nextcloud / android

📱 Nextcloud Android app
https://play.google.com/store/apps/details?id=com.nextcloud.client
GNU General Public License v2.0
4.27k stars 1.77k forks source link

Warning about possible incompatibility to ownCloud #188

Closed tobiasKaminsky closed 8 years ago

tobiasKaminsky commented 8 years ago

https://github.com/nextcloud/android/pull/141#issuecomment-236938290 @Spacefish

I would display a warning during the initial account adding inside the app, which tells the user, that ownCloud might be incompatible and that he may want to upgrade his server to next cloud.. however we loose all the users which may use the next cloud app but can't control the server installation (cooperate or educational users mainly)

Good idea. Should we, for now, mention the one and only thing (file drop) that is not compatible?

@jancborchardt @LukasReschke

AndyScherzinger commented 8 years ago

I would go for it since it is a neat way to promote the Nc server in the app right in the context of the feature :)

AndyScherzinger commented 8 years ago

pinging @jancborchardt and @LukasReschke again for Feedback

LukasReschke commented 8 years ago

Mhm. Thoughts, @jospoortvliet @karlitschek? I do see that this makes sense for example we have the upload only sharing links which with ownCloud would not work (see https://github.com/nextcloud/android/issues/111)

karlitschek commented 8 years ago

I think this makes sense. Do we have incompatibilities already?

LukasReschke commented 8 years ago

Yes. The files drop upload implemented in https://github.com/nextcloud/android/pull/141

schiessle commented 8 years ago

I think such a warning is OK. I would keep it more general, which makes it also shorter and display space is always a issue on mobile devices. If you wish you could be more precise about the incompatibilities as part of the online documentation.

tobiasKaminsky commented 8 years ago

For this it would be great if /status.php could provide something like "name" set to "nextcloud". When this is absent we can show the warning as a popup warning: Please be aware that the app is designed to be used with Nextcloud. Fully compatibility to ownCloud cannot be guaranteed.

cc @jancborchardt for design/wording.

AndyScherzinger commented 8 years ago

This is a tricky topic. Should we just show this once when login into an oC instance, or do we need to do this on a feature level? My guess would be that not all users are aware of the servers features and might then misinterpred errors or crashed as an app issue which it is not. That might lead to bad votings. Thus I would show say we should show the popup in each context where a function is clicked but not oC supported.

What do you think?

tobiasKaminsky commented 8 years ago

Every time might be too much. Still we do not have a real way to distinguish between oc and nc, or?

AndyScherzinger commented 8 years ago

Don't know. Do you know if there is a way @LukasReschke?

jancborchardt commented 8 years ago

I’d say this should be on a context basis as @AndyScherzinger says. Just like when a file type is not possible to be previewed. It’s only important when you actually want to do that.

»This functionality is not available with your ownCloud server. Update to Nextcloud for the full functionality. :)«

Something like that. Then with buttons to »Close« and »Learn more«, leading to the server install page.

tobiasKaminsky commented 8 years ago

I will do it the way @jancborchardt described for "files drop". It will be a snackbar and there is only one button allowed. I will us it for "Learn more" as the snackbar disappears after ~3s so the "Close" button is not needed.

AndyScherzinger commented 8 years ago

Sounds great, also using the snackbar, which is less invasive :+1:

tobiasKaminsky commented 8 years ago

screenshot_20160905_101444 @jancborchardt the text is too long :/

AndyScherzinger commented 8 years ago

How about »This is a Nextcloud server functionality. Update for the full functionality.« in case this is still to long, let's just go for »This is a Nextcloud server functionality.«

It gives you all the info you need :)

tobiasKaminsky commented 8 years ago

Second idea is still too long. So I am using the very short version. Still it generates a line break on my emulator, but this is device and language specific.. screenshot_20160905_102121

AndyScherzinger commented 8 years ago

True, but I think a line break is fine and like you said this depends on the language. So I'd say go with it :+1:

jancborchardt commented 8 years ago

This is a Nextcloud feature, please update.

How about that?

Also, @tobiasKaminsky in your last screenshot the text is very dark?

tobiasKaminsky commented 8 years ago

Also, @tobiasKaminsky in your last screenshot the text is very dark?

It is plain white text. Maybe it is due to the screenshot

AndyScherzinger commented 8 years ago

@tobiasKaminsky did you oush the code to GH already? :smiley: Just curious.

tobiasKaminsky commented 8 years ago

Not yet. It needed some code for library. I will push it in 4-5 hours.

AndyScherzinger commented 8 years ago

:smiley: :+1:

tobiasKaminsky commented 8 years ago

Now online :)

AndyScherzinger commented 8 years ago

@jancborchardt this is due to the screenshot, see mine: unbenannt