-
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Comments
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 :) |
pinging @jancborchardt and @LukasReschke again for Feedback |
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) |
I think this makes sense. Do we have incompatibilities already? |
Yes. The files drop upload implemented in #141 |
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. |
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: cc @jancborchardt for design/wording. |
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? What do you think? |
Every time might be too much. |
Don't know. Do you know if there is a way @LukasReschke? |
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. |
I will do it the way @jancborchardt described for "files drop". |
Sounds great, also using the snackbar, which is less invasive 👍 |
|
How about It gives you all the info you need :) |
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 👍 |
How about that? Also, @tobiasKaminsky in your last screenshot the text is very dark? |
It is plain white text. Maybe it is due to the screenshot |
@tobiasKaminsky did you oush the code to GH already? 😃 Just curious. |
Not yet. |
😃 👍 |
Now online :) |
@jancborchardt this is due to the screenshot, see mine: |
#141 (comment) @Spacefish
Good idea.
Should we, for now, mention the one and only thing (file drop) that is not compatible?
@jancborchardt @LukasReschke
The text was updated successfully, but these errors were encountered: