nextcloud / nextcloud/server

Comments within a federated share should be made visible to all federated parties

Open
#2,197 5 comments 6 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

1. to develop design enhancement feature: comments feature: federation feature: sharing
Dominant language
PHP
Stars
36.9k
Forks
5.2k
Avg merge
2d 3h
Merged PRs (30d)
713

Description

In the context of a (federated) share, we consider comments as a valuable means of communication and hence an essential part of the collaboration effort.

While we appreciate use cases where tags are being contained to the server where they are initially created and stored, we deem it appropriate to make comments within a federated share available to all parties having access to that share.

Steps to reproduce
  1. For collaboration purposes, UserA federates a share (shareX) on serverA to UserB on ServerB ;
  2. UserA creates comments in the shareX realm ;
  3. UserB creates comments in the shareX realm.
Expected behaviour

Parties with access to shareX (i.e. both UserA and UserB) should see all comments in this share's realm.

Actual behaviour

UserA only sees his/her own shareX comments (stored on ServerA), UserB only sees his/her own shareX comments (stored on ServerB).

Server configuration

Nextcloud version: 10.0.1

Workflow proposal :
  • Add a 'public/private' boolean field to table oc_comments (defaulting to private) ;

  • When a share is federated, disclose a [public] checkbox next to the [Post] button in the UI, allowing (new) comments to be made public ;

  • For performance and privacy reasons, comments which are tagged public are stored in or transferred to table oc_comments on the server hosting the share, private comments are stored on or transferred to the user's server;

  • Reversal from public to private status and vice versa can only be executed by the comment owner [oc_comments.actor_id].

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by tracing how federated shares create, store, and display comments on both servers, then inspect the oc_comments schema and the comment UI around the Post button. Done means parties with access to a federated share can see the intended comments, with public/private behavior and ownership rules clarified and covered by regression tests.

Written by the indexing model from the issue text.

Assessment

Tech stack
php
Domain
backend, databases, distributed-systems, frontend
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.