-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Comment mention autosuggestion #2964
Comments
Hi, can I do this issue? |
Yes, you're welcome to do that. |
... or similar to github or gitlab when mentioning somebody in e.g. this issue here. Especially, when social authentication is activated (e.g. via auth0), and e.g. two users are registering with the same first name, this results into:
In the second case, there is no chance to remember the user name, hence autosuggestion is kind of a must. |
@SaptakS When touching the editor, please keep in mind that we want more features there in future. Related issues I've remembered now:
|
This could probably use the /api/users/. It should be exteded to support this use case. The django_filters can be added to add filtering and add support for making the API call by non superusers by filtering the available fields to them (probably just username and full_name should be exposed in this case). |
* Adds user suggestions in comment section * Uses codemirror and show-hint addon to show autocomplete users * Use markdown mode with show hint over multiple different textarea * Adds API and selenium tests for user mentions * Modify some logics for showing user suggestions * Using async method for showing hint Fixes #2964 Co-authored-by: Michal Čihař <[email protected]>
Thank you for your report, the issue you have reported has just been fixed.
|
Is your feature request related to a problem? Please describe.
When mentioning someone in the comments section it would be helpful to be able to type and select them using a handle, similar to the system used in Discord.
Describe the solution you'd like

Describe alternatives you've considered
//
Additional context
//
The text was updated successfully, but these errors were encountered: