Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix Hyperlink without Remove #727

Merged
merged 2 commits into from
Feb 8, 2022
Merged

fix Hyperlink without Remove #727

merged 2 commits into from
Feb 8, 2022

Conversation

sml2
Copy link

@sml2 sml2 commented Jan 9, 2022

I tried to fix #726 , but I'm not sure if it is reasonable or not if I need to associate the Remove method.

@tonyqus tonyqus added this to the NPOI 2.5.7 milestone Jan 10, 2022
@tonyqus tonyqus added the bug label Jan 10, 2022
@tonyqus
Copy link
Member

tonyqus commented Feb 4, 2022

Please follow https://svn.apache.org/viewvc?view=revision&revision=1896552 to write a unit test for this fix.

@sml2
Copy link
Author

sml2 commented Feb 5, 2022

My solution can not run test,Cloud you help me

@tonyqus tonyqus changed the base branch from master to XlsFixes February 8, 2022 20:45
@tonyqus tonyqus merged commit 73512b9 into nissl-lab:XlsFixes Feb 8, 2022
@tonyqus
Copy link
Member

tonyqus commented Mar 19, 2023

This is POI 52447

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unable to Open Excel XLS with hyperlink cells
2 participants