Skip to content

Self mentions are allowed during edition but not in final comment #7254

Description

@danxuliu

Self mentions behaviour should be unified, either by disabling them during edition or by enabling them in final comments.

@blizzz

comment-self-mention

Activity

  1. added this to the Nextcloud 13 milestone on Nov 23, 2017
  2. blizzz commented on Nov 27, 2017

    @blizzz
    Member

    they should be filtered out. That's why it does not work in the final comments. Github does not list offer yourself for completion, however does it when rendering. Exactly vice-versa to what we have now ;) Turning it around does not since you don't (necessarily) know your userid. So I guess we should filter it out, instead of allowing it, to follow our original path. Confirmed, @jancborchardt ?

  3. jancborchardt commented on Nov 30, 2017

    @jancborchardt
    Member

    Yeah, yourself should of course be filtered out of the suggestions, just like in sharing.

    However it's fine to show it markupped when you mention yourself in the finalized comment (maybe also when you do it while writing, but not if it's extra work to check).

  4. self-assigned this
    on Dec 4, 2017
  5. nickvergessen commented on Jan 11, 2018

    @nickvergessen
    Member

    Well if it was allowed, you could click on the user name there and get teh user info popup which allows you to email, talk, ... people on the instance, so I could see usecases for it.

  6. removed this from the Nextcloud 13 milestone on Jan 11, 2018
  7. jancborchardt commented on Jan 12, 2018

    @jancborchardt
    Member

    @nickvergessen yeah, this is about self-mentions though – and mailing or Talk-ing to yourself is not really a thing? ;)

    When you click on your own name/avatar, the dropdown options should rather be show as "It's you" and "Edit profile". cc @georgehrke

  8. added this to the Nextcloud 14 milestone on Jan 12, 2018
  9. nickvergessen commented on Jan 12, 2018

    @nickvergessen
    Member

    Maybe I misunderstood, I thought the problem is that for everyone the dropdown is not rendered if author === mentioned.
    Or is it dropdown not rendered if mentioned === current user?

  10. danxuliu commented on Jan 15, 2018

    @danxuliu
    MemberAuthor

    @nickvergessen

    I thought the problem is that for everyone the dropdown is not rendered if author === mentioned.

    You are right, self mentions are not rendered either when seen by other users, as if author === mentioned the mention is filtered by the server.

    Or is it dropdown not rendered if mentioned === current user?

    Originally I was referring to the inconsistent behaviour in this case; in any case I agree that self mentions should be rendered at least when seen by a different user.

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

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions