Merge email and remote sharing
Nobody has claimed this yet.
- Dominant language
- PHP
- Stars
- 36.9k
- Forks
- 5.2k
- Avg merge
- 2d 3h
- Merged PRs (30d)
- 713
Description
To prevent showing an email-adress (or Federated Cloud ID) twice in the sharing with user dropdown, I'd propose to merge them into one single "admin@cloud.example.com"-entry ... Clicking on it will first check whether there is an Federation server (remote, prioritized), and if not - It will use email as fallback method (because it makes no sense to share with somebody who as Nextcloud himself via email).
Would fix messages like:
Currently:
- admin@cloud.example.com (remote)
- admin@cloud.example.com (email)
What do you think? @schiessle @jancborchardt
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
The issue provides no file or test names. Begin by tracing the sharing-with-user dropdown and the remote/email selection flow described in the body, then verify that one displayed address chooses federation first and email only as a fallback without duplicate entries.
Written by the indexing model from the issue text.
Assessment
- Domain
- backend, frontend
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100