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.

Sign up for GitHub

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

Jump to bottom

Edit/delete comments #893

Closed
9 tasks done
ggwadera opened this issue Jun 23, 2021 · 0 comments · Fixed by #2278
Closed
9 tasks done

Edit/delete comments #893

ggwadera opened this issue Jun 23, 2021 · 0 comments · Fixed by #2278
Assignees
@HS-90
Labels
enhancement New feature or request UI User Interface

Comments

@ggwadera
Copy link
Collaborator

ggwadera commented Jun 23, 2021 •
edited by HS-90
Loading

Currently we can only add line comments while reviewing. It would be nice if we could also edit and/or delete a comment.

Requirements

  • Ability to edit comments
    • Mutation and resolver on the back-end
    • UI for the front-end
  • Ability to delete comments
    • Mutation and resolver on the back-end
    • UI for the front-end
  • Only the comment owner must be able to edit/delete a comment
    • Edit/delete options should not be rendered on a comment the user doesn't own
    • The backend should also check if the user owns the comment by the user id on the session.
@ggwadera ggwadera added enhancement New feature or request good first issue Good for newcomers UI User Interface labels Jun 23, 2021
@ggwadera ggwadera removed the good first issue Good for newcomers label Jul 2, 2021
@kondanna kondanna self-assigned this Nov 23, 2021
@flacial flacial self-assigned this Feb 3, 2022
This was referenced Feb 8, 2022
Merged
Merged
Merged
Merged
@flacial flacial mentioned this issue Feb 15, 2022
Merged
@HS-90 HS-90 self-assigned this Jul 24, 2022
This was referenced Sep 3, 2022
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@HS-90 HS-90

Labels
enhancement New feature or request UI User Interface
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Edit Comments Frontend Functionality HS-90/c0d3-app
4 participants
@ggwadera @kondanna @flacial @HS-90