Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FEATURE] Enabled User @mentions and @mention-filters in core editor package #2544

Merged
merged 47 commits into from
Nov 1, 2023

Conversation

henit-chobisa
Copy link
Collaborator

@henit-chobisa henit-chobisa commented Oct 26, 2023

Description

The current pull request enables the usage of user-mentions in the tiptap editor, with features such as self highlighting, redirection and sending notifications from the backend.
image

Features

Sending Notification on being mentioned with context of being mentioned

Irrespective of a person being a subscriber of the issue or not, if he's mentioned in a particular issue, he will be sent a notification for being mentioned, along with that he will be added as a subscriber to the issue, to receive further updates on the issue. For testing the possible set of cases and scenarios please go through https://henit.notion.site/feat-mentions-c96dc6cfe53a4e2eafb0acb252cb80c4?pvs=4

image

Redirecting on the user profile, when clicked on the mention

The mention acts as a button, hence if you have provided the redirect_uri in the underlying data structure IMentionSuggestion[], you can see it redirecting to the user's profile. In future we can use this for redirecting to specific sets of views such as cycles or issues itself.

Screen.Recording.2023-10-03.at.5.36.15.PM.mov

Dynamic Highlights of entities in an issue

The mention feature also come with highlightin, which makes it extremely easy to spot a particular highlight, which can be anything from highlighting the current user from the content or highlighting a set of issues from a query of "Show me all the issues where a particular set of issues are mentioned". Currently the highlighting works only in the issues/form.tsx, and in side-peak-view.
image

For Example: : If the highlight group is for the current user, which means that in every issue, if the current user is mentioned he must be highlighted, in this case it's dynamic because for an issue where @crew and @captain both are mentioned, if @crew is signed in, he must be highlighted and so as captain, which makes us dynamically update the values. Below is the picture of the implementation, which show how dynamic highlights work for two different highlight groups.

Issue View of the Captain
image

Issue View for the Crew
image

Issue filters with Embedded Mentions

Many of the times, customers tend to search for the issues where they are mentioned or maybe a particular issue or a person is mentioned, apps such as slack provide such functionality to quickly grab through text messages, which has been very convenient, hence along with the ability to tag people, I have also added an ability to filter issues according to the mentioned users.

Screen.Recording.2023-10-30.at.6.07.30.PM.mov

@vercel
Copy link

vercel bot commented Oct 26, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

6 Ignored Deployments
Name Status Preview Comments Updated (UTC)
plane ⬜️ Ignored (Inspect) Visit Preview Nov 1, 2023 11:05am
plane-dev ⬜️ Ignored (Inspect) Visit Preview Nov 1, 2023 11:05am
plane-sh ⬜️ Ignored (Inspect) Visit Preview Nov 1, 2023 11:05am
plane-sh-dev ⬜️ Ignored (Inspect) Visit Preview Nov 1, 2023 11:05am
plane-sh-stage ⬜️ Ignored (Inspect) Visit Preview Nov 1, 2023 11:05am
plane-staging ⬜️ Ignored (Inspect) Visit Preview Nov 1, 2023 11:05am

@henit-chobisa henit-chobisa changed the title [FEATURE] Enabled User @mentions in tiptap editor component [FEATURE] Enabled User @mentions and @mention-filters in core editor package Oct 26, 2023
@henit-chobisa henit-chobisa self-assigned this Oct 26, 2023
apiserver/requirements/base.txt Outdated Show resolved Hide resolved
apiserver/plane/db/models/issue.py Outdated Show resolved Hide resolved
apiserver/plane/api/views/issue.py Outdated Show resolved Hide resolved
@sriramveeraghanta sriramveeraghanta merged commit d511799 into develop Nov 1, 2023
8 of 12 checks passed
@sriramveeraghanta sriramveeraghanta deleted the feat/editor-mentions branch November 1, 2023 11:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants