weixin_39531834
2021-01-08 17:51 阅读 1

Hyperlink Support

Adaptation of a previous version PR https://github.com/box/spout/pull/352

I found the pull request for a previous version while seeing if native Hyperlink support was added. I personally don't need any other "formulas" just the HYPERLINK to have clickable cells to external websites. Not sure if this will be useful but it is just added the regex and xml from the above PR. First ever PR so not sure if I am doing this right, but a small change if anyone needs hyperlinks like me.

该提问来源于开源项目:box/spout

  • 点赞
  • 写回答
  • 关注问题
  • 收藏
  • 复制链接分享

5条回答 默认 最新

  • weixin_39902107 weixin_39902107 2021-01-08 17:51

    Hi , thanks for the pull request. Before we can merge it, we need you to sign our Contributor License Agreement. You can do so electronically here: http://opensource.box.com/cla

    Once you have signed, just add a comment to this pull request saying, "CLA signed". Thanks!

    点赞 评论 复制链接分享
  • weixin_39531834 weixin_39531834 2021-01-08 17:51

    Hi , thanks for the pull request. Before we can merge it, we need you to sign our Contributor License Agreement. You can do so electronically here: http://opensource.box.com/cla

    Once you have signed, just add a comment to this pull request saying, "CLA signed". Thanks!

    CLA Signed (if this is a valid PR)

    点赞 评论 复制链接分享
  • weixin_39902107 weixin_39902107 2021-01-08 17:51

    Verified that has just signed the CLA. Thanks, and we look forward to your contribution.

    点赞 评论 复制链接分享
  • weixin_40003451 weixin_40003451 2021-01-08 17:51

    Hi ! Thanks for submitting your first PR. We try to have the same features for XLSX and ODS files. So I can't accept this PR as is (+ broken/missing tests). Anyway, I'll leave it open so that people can use what you did in their project/fork.

    点赞 评论 复制链接分享
  • weixin_39605414 weixin_39605414 2021-01-08 17:51

    Hi ! Thanks for submitting your first PR. We try to have the same features for XLSX and ODS files. So I can't accept this PR as is (+ broken/missing tests). Anyway, I'll leave it open so that people can use what you did in their project/fork.

    I think this feature also very good. Can you describe what still needs to be done so that this PR is accepted. How must the hyperlink syntax look like that it fits?

    Another possibility would be not only to set the cell values from outsite Means other (xlsx, ods) xml elements.

    点赞 评论 复制链接分享

相关推荐