Skip to content

docs: add missing comment history panel actions#1099

Open
Aamod007 wants to merge 4 commits into
layer5io:masterfrom
Aamod007:fix-issue-549-comment-history
Open

docs: add missing comment history panel actions#1099
Aamod007 wants to merge 4 commits into
layer5io:masterfrom
Aamod007:fix-issue-549-comment-history

Conversation

@Aamod007

@Aamod007 Aamod007 commented Jun 14, 2026

Copy link
Copy Markdown
Contributor

Summary

This PR updates the Kanvas designer documentation to reflect the latest capabilities available within the comment history panel, ensuring the docs accurately mirror the product's features.

Changes Included

  • index.md (Comments section):
    • Added detailed descriptions for managing threads directly from the history panel, including resolving comments, muting notifications, and deleting comments.
    • Added a best-practice note clarifying that resolving a comment is generally preferred over deleting it, as it preserves historical context.
    • Updated the specific sections on resolving, muting, and deleting to explicitly mention that these actions can also be accessed via the comment history panel.
    • Fixed a minor spelling typo (contructive -> constructive).
  • feature_data.json:
    • Appended "Delete individual comments" to the feature list under the Design Review Notifications configuration to reflect the complete set of capabilities.

Screenshot

Screenshot 2026-06-14 144652

Issue Linked

Closes #549

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the Kanvas designer comments documentation and feature data to detail new capabilities, such as managing individual comment threads, muting notifications, and deleting individual comments from the comment history panel. It also fixes a typo ('contructive' to 'constructive'). The review feedback suggests improving terminology consistency by referring to resolving 'comment threads' rather than 'individual comments' and clarifying that deleting individual comments removes the specific comments rather than the entire thread.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread content/en/kanvas/designer/comments/index.md Outdated
Comment thread content/en/kanvas/designer/comments/index.md Outdated
@github-actions

github-actions Bot commented Jun 14, 2026

Copy link
Copy Markdown
Contributor

🚀 Preview deployment: https://docs.layer5.io/pr-preview/pr-1099/

Note: Preview may take a moment (GitHub Pages deployment in progress). Please wait and refresh. Track deployment here

@NSTKrishna

NSTKrishna commented Jun 14, 2026

Copy link
Copy Markdown
Contributor

@Aamod007 There is one commit incorrectly signed off. fix it

@Aamod007 Aamod007 force-pushed the fix-issue-549-comment-history branch from 8a01dbf to f56fc81 Compare June 14, 2026 10:51
Aamod007 and others added 3 commits June 14, 2026 16:24
Signed-off-by: Aamod007 <aamodkumar2006@gmail.com>
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Signed-off-by: Aamod Kumar <119789532+Aamod007@users.noreply.github.com>
Signed-off-by: Aamod007 <aamodkumar2006@gmail.com>
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Signed-off-by: Aamod Kumar <119789532+Aamod007@users.noreply.github.com>
Signed-off-by: Aamod007 <aamodkumar2006@gmail.com>
@Aamod007 Aamod007 force-pushed the fix-issue-549-comment-history branch from f56fc81 to c123805 Compare June 14, 2026 10:54
@Aamod007

Copy link
Copy Markdown
Contributor Author

@Aamod007 There is one commit incorrectly signed off. fix it

@NSTKrishna Fixed! All 3 commits have been rebased and signed off. The DCO check should be passing now!

@Aamod007

Copy link
Copy Markdown
Contributor Author

@NSTKrishna can u have a look at this PR too,PR #1100

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.

[Docs] Add missing actions to comment history panel

2 participants