Skip to content

Share chart data implementation between XSSFChart and XSLFChart through XDDF#68

Closed
Alain-Bearez wants to merge 28 commits intoapache:trunkfrom
cuali:xddf-usermodel
Closed

Share chart data implementation between XSSFChart and XSLFChart through XDDF#68
Alain-Bearez wants to merge 28 commits intoapache:trunkfrom
cuali:xddf-usermodel

Conversation

@Alain-Bearez
Copy link
Copy Markdown
Contributor

The main goal of this work is to be able to:

What is left to be done:

  • complete implementation of first five chart types;
  • implement other chart types.

@aperrot42
Copy link
Copy Markdown

A shame I decided to go with docx4j to implement a higher level charting library for pptx... We would have had a lot of common code to share.

@asfgit
Copy link
Copy Markdown

asfgit commented Sep 28, 2017

Can one of the admins verify this patch?

Copy link
Copy Markdown
Contributor

@onealj onealj left a comment

Choose a reason for hiding this comment

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

Skimming through the changes, looks good to me. I think this is a good idea!

@Alain-Bearez
Copy link
Copy Markdown
Contributor Author

@onealj After this pull request will have been merged, I suggest you continue your review on the next PR related to XDDF: #72

The third PR about XDDF is not ready yet.

@onealj
Copy link
Copy Markdown
Contributor

onealj commented Nov 3, 2017

This PR looks good and ready to merge. We should probably have you sign a contributor license agreement before merging. I sent an email to your cua.li address with details.

@centic9
Copy link
Copy Markdown
Member

centic9 commented Nov 3, 2017

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.

5 participants