### Author:  [@anjisvj](https://sef.discourse.group/u/anjisvj)
#### Post date: [January 12, 2022, 7:17am UTC](https://sef.discourse.group/t/implementing-the-comments-adding-feature/377/2 "2022-01-12T07:17:30Z")
**Backtend**
Here’s the draft PR along with the new comments table. The relationship is `A profile can add many comments on a mentee`. @piumal1999 @Gravewalker @jaye Can I have some feedback on this?
>
>
> \## Purpose
>
> The purpose of this PR is to fix #282
>
> \## Goals
>
>
> \## Approac…h
>
>
> \### Screenshots
>
>
> \## Checklist
> \- \[x\] This PR doesn't commit any keys, passwords, tokens, usernames, or other secrets.
> \- \[x\] I have read and understood the development best practices guidelines ( http://bit.ly/sef-best-practices )
> \- \[x\] My code follows the style guidelines of this project
> \- \[x\] I have performed a self-review of my own code
> \- \[\] I have commented my code, particularly in hard-to-understand areas
> \- \[\] I have made corresponding changes to the documentation
>
> \## Related PRs
>
>
> \## Test environment
>
>
> \## Learning
**Frontend**
I’m halfway into integrating the APIs with the admin UI
---
### Author:  [@jaye](https://sef.discourse.group/u/jaye)
#### Post date: [January 12, 2022, 8:04am UTC](https://sef.discourse.group/t/implementing-the-comments-adding-feature/377/3 "2022-01-12T08:04:59Z")
Thanks for the update @anjisvj!
You are heading on the right path. Please add `created_at` field to the comment table. So that we can know when they commented. You could rename the profile\_id field into something more readable. ex: commented\_by, commented\_profile\_id
Regarding the UI, we don’t need to display two sections, Comments by mentor and comments by admin. Let’s display all of them using the [antd’s list component](https://ant.design/components/list/) with their profile picture, name and time. (We could show their role later)
@piumal1999 @Gravewalker What do you think?
---
### Author:  [@anjisvj](https://sef.discourse.group/u/anjisvj)
#### Post date: [January 12, 2022, 8:07am UTC](https://sef.discourse.group/t/implementing-the-comments-adding-feature/377/4 "2022-01-12T08:07:57Z")
Thanks for the feedback! I’ll update the columns.
Regarding the UI your way is perfect. I was struggling with the UI. xD
---
### Author:  [@anjisvj](https://sef.discourse.group/u/anjisvj)
#### Post date: [January 12, 2022, 1:54pm UTC](https://sef.discourse.group/t/implementing-the-comments-adding-feature/377/5 "2022-01-12T13:54:15Z")
Here the Admin UI
Are we going to show the admin comments to the mentor? Are we going to enable comments throughout the program?
@jaye @Dharana_J
---
### Author:  [@jaye](https://sef.discourse.group/u/jaye)
#### Post date: [January 13, 2022, 5:38am UTC](https://sef.discourse.group/t/implementing-the-comments-adding-feature/377/6 "2022-01-13T05:38:22Z")
Let’s make it visible to both parties. So that they can use it as a communication method. But we should make sure to indicate to users that comments are visible to other admins and relevant mentors. We can add a visibility feature later enabling them to add private comments (ex: only me, admins only, …)
For the design, I would suggest something like the. following: (Only if we have sufficient time)
To display that time ago thing, there’s a tiny library named [javascript-time-ago](https://www.npmjs.com/package/javascript-time-ago)
---
### Author:  [@anjisvj](https://sef.discourse.group/u/anjisvj)
#### Post date: [January 13, 2022, 7:44am UTC](https://sef.discourse.group/t/implementing-the-comments-adding-feature/377/7 "2022-01-13T07:44:19Z")
Thanks for the suggestion. @jaye
Made the changes.
---
### Author:  [@jaye](https://sef.discourse.group/u/jaye)
#### Post date: [January 13, 2022, 8:02am UTC](https://sef.discourse.group/t/implementing-the-comments-adding-feature/377/8 "2022-01-13T08:02:40Z")
Awesome!
The comments can be deleted only by the one who commented.
A suggestion:
It would be nice if we could move the delete option to the view more menu shown in the example I provided. Later we can add the edit option to the same menu. (The menu icon should only be displayed in front of their own comments)
---
### Author:  [@anjisvj](https://sef.discourse.group/u/anjisvj)
#### Post date: [January 13, 2022, 12:36pm UTC](https://sef.discourse.group/t/implementing-the-comments-adding-feature/377/9 "2022-01-13T12:36:59Z")
I made the requested changes.
The PRs are ready for review:
>
>
> \## Purpose
>
> \- The purpose of this PR is to fix #293
>
> \## Goals
>
> \- Mentors… and Admins should be able to add/delete comments
>
> \## Approach
>
> \- Create the comments section component
>
> \## Screenshots
>
> !\[image\](https://user-images.githubusercontent.com/43912578/149330465-0db7a3ce-2327-4bf9-aa97-ed01222f690f.png)
>
> \## Checklist
> \- \[x\] This PR doesn't commit any keys, passwords, tokens, usernames, or other secrets.
> \- \[x\] I have read and understood the development best practices guidelines ( http://bit.ly/sef-best-practices )
> \- \[x\] My code follows the style guidelines of this project
> \- \[x\] I have performed a self-review of my own code
> \- \[\] I have commented my code, particularly in hard-to-understand areas
> \- \[\] I have made corresponding changes to the documentation
>
>
> \## Purpose
>
> \- The purpose of this PR is to fix #282
>
> \## Goals
>
> \- Mentors …and Admins should be able to add/delete comments
> \## Approach
>
> \- Create CRUD enpoints
> \- Create comment model, service and repository
>
> \## Checklist
> \- \[x\] This PR doesn't commit any keys, passwords, tokens, usernames, or other secrets.
> \- \[x\] I have read and understood the development best practices guidelines ( http://bit.ly/sef-best-practices )
> \- \[x\] My code follows the style guidelines of this project
> \- \[x\] I have performed a self-review of my own code
> \- \[\] I have commented my code, particularly in hard-to-understand areas
> \- \[\] I have made corresponding changes to the documentation