-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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 function to remove or update comment in a cell #849
Comments
when we try to add comment to a cell with comment, when open the excel file, the excel program will complain about 'there is an error occur, but we has recovery from it' |
Hi @coreyzzp, thanks for your feedback, could you provide any code and attachments to reproduce this issue? |
func TestCreateExcelAndUpdateComment(t *testing.T) {
target := "test.xlsx"
target2 := "test-after.xlsx"
{
var err error
fd := excelize.NewFile()
err = fd.AddComment("Sheet1", "A1", fmt.Sprintf(`{"author":"Excelize: ","text":"This is a comment" }`))
require.Nil(t, err)
err = fd.SaveAs(target)
require.Nil(t, err)
// here open test.xlsx is find
}
{
fd, err := excelize.OpenFile(target)
require.Nil(t, err)
err = fd.AddComment("Sheet1", "A1", fmt.Sprintf(`{"author":"Excelize: ","text":"hello world"}`))
require.Nil(t, err)
err = fd.SaveAs(target2)
require.Nil(t, err)
// now open test-after.xlsx an error popup
}
} open test-after.xlsx |
it will be good to has a method to delete or update comment in a cell |
This lib doesn't support modify comment and set the comment repeatedly on one cell currently. I've add this feature support in the Roadmap. |
thx~ |
find an article about "comments" and "notes": The difference between threaded comments and notes |
[WeOpen Star] I would like to help |
1 similar comment
[WeOpen Star] I would like to help |
…omment (qax-os#1317) - Update unit tests for the delete comment - Add 3 errors function for error messages
…omment (qax-os#1317) - Update unit tests for the delete comment - Add 3 errors function for error messages
…omment (qax-os#1317) - Update unit tests for the delete comment - Add 3 errors function for error messages
…omment (qax-os#1317) - Update unit tests for the delete comment - Add 3 errors function for error messages
No description provided.
The text was updated successfully, but these errors were encountered: