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

Question about additional functionality #15

Open
matsuobasho opened this issue Feb 19, 2025 · 2 comments
Open

Question about additional functionality #15

matsuobasho opened this issue Feb 19, 2025 · 2 comments

Comments

@matsuobasho
Copy link

I find the package very useful. Enquiring about a more nuanced feature.

I have a document with tracked changes. I want to make additional changes on top of that document. Then after feeding both these documents to redlines, I’d like the output to have both original tracked changes and the new ones.

Is this possible and if so, how would I set it up?

Realize it may be difficult to implement. If that’s the case it would be useful to know what the major limitations for that are.

@JSv4
Copy link
Owner

JSv4 commented Feb 22, 2025

Hey @matsuobasho, yes, this is definitely possible with the current underlying engine (OpenXMLPowertools). One day 🙄, I will find the time to start to write a new engine. However, the amount of engineering that's gone into the C# OpenXMLPowertools version I've been wrapping is pretty impressive and it's probably most economical to start maintaining it and get more mileage out of its author's years of work and expertise. If you look at one of the examples in C#, the actual ability to consolidate changes from multiple docx exists. Can't vouch for the accuracy, but we could definitely expose the underlying engine's feature so at least it's available. If you want to see how I wrote the existing wrapper and try to add it via a PR, I would welcome the help! I will definitely review and merge, if appropriate. If you're waiting for me to do this, probably going to take a while tbh. Pretty tied up with other things.

@matsuobasho
Copy link
Author

@JSv4 thanks for the feedback. Totally understand and appreciate you're tied up now. So your recommendation is to take a look at the example you linked. Then you mentioned seeing how you wrote the existing wrapper and integrating that C# example there? If so, please share code for the existing wrapper.

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

No branches or pull requests

2 participants