Skip to content

Comments: Replace CommentEntityEvent with a registration-bases solution within CommentsManager. #1539

Closed

Description

@blizzz

Currently, adding DAV support to other than 'files' type of comments, a dispatcher is being used to register a closure, which is handled by the comments RootCollection.

The CommentsManager should take this over and offer a possibility to register SubCollections as such. The RootCollection would query it let the corresponding instance continue.

The new API and switch from the Dispatcher shall be communicated with the release notes and possibly on Discourse.

As discussed with @nickvergessen while working at #1449

Activity

  1. added this to the Nextcloud 11.0 milestone on Sep 27, 2016
  2. self-assigned this
    on Sep 27, 2016
  3. removed this from the Nextcloud 12.0 milestone on Apr 26, 2017
  4. added
    1. to developAccepted and waiting to be taken care of
    and removed
    staleTicket or PR with no recent activity
    on Jun 12, 2019
  5. szaimen commented on May 21, 2021

    @szaimen
    Contributor

    I suppose this issue is still valid? If not, please close this issue!

  6. nickvergessen commented on May 21, 2021

    @nickvergessen
    Member

    It not has those dedicated event. Sounds good enough for me atm. What do you think @blizzz

  7. removed their assignment
    on Jul 12, 2023
  8. blizzz commented on Jul 12, 2023

    @blizzz
    MemberAuthor

    if there is no need, we can just close it, instead of keeping it open eternally. can always reopen.

  9. nickvergessen commented on Jul 13, 2023

    @nickvergessen
    Member

    PR at #39304 cough

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions