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.

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

Add AddComment method to handle cell comments in Worksheets #15

Merged
merged 1 commit into from
Apr 15, 2024

Conversation

fahadadeel
Copy link
Collaborator

Implement the AddComment method in the Worksheet class to add or update comments in specific cells. This method ensures compatibility with existing comments, managing the worksheet's comment part and authors list dynamically. It supports adding new comments if none exist or updating existing ones, thereby enhancing the functionality of our Excel handling utility.

This update is crucial for features that rely on providing feedback or notes directly within the Excel documents, such as automated report generation or data validation processes.

  • Ensure the worksheet comments part exists before adding or updating comments.
  • Manage authors list to avoid duplicates and use existing indices where possible.

Implement the AddComment method in the Worksheet class to add or update comments in specific cells. This method ensures compatibility with existing comments, managing the worksheet's comment part and authors list dynamically. It supports adding new comments if none exist or updating existing ones, thereby enhancing the functionality of our Excel handling utility.

This update is crucial for features that rely on providing feedback or notes directly within the Excel documents, such as automated report generation or data validation processes.

- Ensure the worksheet comments part exists before adding or updating comments.
- Manage authors list to avoid duplicates and use existing indices where possible.
@fahadadeel fahadadeel merged commit f08135d into fileformat-cells:main Apr 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant