nextcloud / nextcloud/notes-android
Support `readonly` notes
Nobody has claimed this yet.
- Dominant language
- Java
- Stars
- 1.1k
- Forks
- 168
- Avg merge
- 1d 17h
- Merged PRs (30d)
- 31
Description
Is your feature request related to a problem? Please describe.
For example when you share a note (via file sharing, as the direct share functionality isn't implemented yet #231) with restricted permissions.
If the read-only user tries to open and edit it within the Android app, it will stay stuck on "Synchronization failed", he then won't ever be able to re-sync it again.
Even the "Cancel" option doesn't work, it looks like it's still trying to push something (shouldn't this action affect only the local storage?).
Describe the solution you'd like
As with the web view, prevent Edit view on notes without write permission
Describe alternatives you've considered
It would be also be very beneficial imo to fix the "Cancel" action, I believe this should act like a "git checkout" and simply retrieve the server's file version?
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by tracing the Android app's note-opening and edit flow for notes with restricted permissions, then inspect how the Synchronization failed and Cancel actions handle pending changes. Compare the behavior with the web view described in the issue. Done means read-only notes cannot enter Edit view and the affected synchronization flow has clear, verified behavior.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- android, java
- Domain
- mobile
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 38/100