-
-
Notifications
You must be signed in to change notification settings - Fork 601
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
Comments component #2564
Comments component #2564
Conversation
@sinejespersen FYI I am working on centralizing the comments function into one central component. Your edit comment functionality would be great for this PR as well. Maybe we can work on it together. |
You are welcome to merge the comment pr, if that is what you mean? |
Link to ticket
Please add a link to the GitHub issue being addressed by this change.
Description
Please include a short description of the suggested change and the reasoning behind the approach you have chosen.
Screenshot of the result
If your change affects the user interface, you should include a screenshot of the result with the pull request.
Checklist
If your code does not pass the requirements on the checklist, you should add a comment explaining why this change
should be exempt from the list.
Additional comments or questions
If you have any further comments or questions for the reviewer, please add them here.