-
Notifications
You must be signed in to change notification settings - Fork 91
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
Fork ? alternative ? #101
Comments
@bflorat - did you find alternatives after you posted? |
@crgardner No but I didn't actively searched. Note that I had to fork this project to fix #32. I forked this project to fix this. It is available on Maven central at
Let me know if you find an alternative. |
I got rid of all that index stuff in the codebase. You can just delete the object by getting the parent and then calling remove on it and pass in the object you want to remove. It greatly simplified the codebase. |
Thanks. I don't remember well but it must be more or less what I did here I also restored the maven packaging, I don't use gradle. |
My understanding is that newer java 11 docx4j rely on the jee jarkata
packages but stamper expects javax objects. I was wanting to see if stamper
could be updated to use the newer docx4j versions and hence be able to deal
either Jakarta packages.
…On Wed, Mar 23, 2022, 5:11 PM dallanmc ***@***.***> wrote:
I got rid of all that index stuff in the codebase. You can just delete the
object by getting the parent and then calling remove on it and pass in the
object you want to remove. It greatly simplified the codebase.
—
Reply to this email directly, view it on GitHub
<#101 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAP3QSGK223QJVZA3G7BPJ3VBOCHZANCNFSM5NAA57NQ>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
I'm interested in an active fork as well, we are getting more and more invested in this piece of code at my workplace. If not, i believe we will do it ourselves anyway. |
I'd like to but I for one have absolutely not the time to do it seriously. Let us know if you start a fork, I may send MR. |
We are starting to actively fork the repo |
This is great - I have a good few improvements that I have implemented (they are in the issues section) - but maybe you've implemented some of them already. A couple of useful ones would be the I am sure there is other stuff, but it's probably in the issues section too. |
This is great ! I'll re-send you this PR #46 ASAP to fix the most important issue I have with this project. |
Thanks @dallanmc, you can make the merge requests yourself on https://github.com/verronpro/docx-stamper with maybe a test case, and i'll include them. If not, then i might take them on myself during the coming month. |
Good work everybody ! Does the If it works now, I may try it again. |
I did not try concurrently with the remove feature, but the repeatDocPart works and use only the given object as sub-context for the "doc part" covered by the comment. |
This project seems abandoned by it original author still 2019.
I still thank him for his great piece of code and as an open source project admin myself, I understand very well that this he ows nothing to us.
However, it is paramount for some of my professional projects and I choosed it when very active back in 2018. I suppose I'm not the only one in this case.
Does somebody know an alternative ? think about a serious fork ?
The text was updated successfully, but these errors were encountered: