Skip to content
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.

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

Warning about possible incompatibility to ownCloud #188

Closed
tobiasKaminsky opened this issue Aug 2, 2016 · 24 comments
Closed

Warning about possible incompatibility to ownCloud #188

tobiasKaminsky opened this issue Aug 2, 2016 · 24 comments

Comments

@tobiasKaminsky
Copy link
Member

#141 (comment) @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
Copy link
Member

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 AndyScherzinger added the needs info Waiting for info from user(s). Issues with this label will auto-stale. label Aug 8, 2016
@AndyScherzinger
Copy link
Member

pinging @jancborchardt and @LukasReschke again for Feedback

@LukasReschke
Copy link
Member

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 #111)

@karlitschek
Copy link
Member

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

@LukasReschke
Copy link
Member

Yes. The files drop upload implemented in #141

@schiessle
Copy link
Member

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
Copy link
Member Author

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.

@tobiasKaminsky tobiasKaminsky added this to the Nextcloud App 1.3.0 milestone Aug 11, 2016
@AndyScherzinger
Copy link
Member

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
Copy link
Member Author

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

@AndyScherzinger
Copy link
Member

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

@jancborchardt
Copy link
Member

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 tobiasKaminsky changed the title Warning about possible incombatibility to ownCloud Warning about possible incompatibility to ownCloud Aug 12, 2016
@tobiasKaminsky
Copy link
Member Author

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
Copy link
Member

Sounds great, also using the snackbar, which is less invasive 👍

@AndyScherzinger AndyScherzinger mentioned this issue Sep 4, 2016
2 tasks
@tobiasKaminsky
Copy link
Member Author

screenshot_20160905_101444
@jancborchardt the text is too long :/

@AndyScherzinger
Copy link
Member

AndyScherzinger commented Sep 5, 2016

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
Copy link
Member Author

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
Copy link
Member

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 👍

@jancborchardt
Copy link
Member

This is a Nextcloud feature, please update.

How about that?

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

@tobiasKaminsky
Copy link
Member Author

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

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

@AndyScherzinger
Copy link
Member

AndyScherzinger commented Sep 5, 2016

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

@tobiasKaminsky
Copy link
Member Author

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

@AndyScherzinger
Copy link
Member

😃 👍

@tobiasKaminsky
Copy link
Member Author

Now online :)

@AndyScherzinger
Copy link
Member

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

@AndyScherzinger AndyScherzinger added enhancement design and removed needs info Waiting for info from user(s). Issues with this label will auto-stale. labels Sep 6, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants