[GitHub] nifi issue #817: NIFI-2380 - Introduce ExtractTNEFAttachments

Github user olegz commented on the issue: https://github.com/apache/nifi/pull/817 @trixpan sorry for the hold up. Actually I would like for @joewitt to look at License/Notice (as you have suggested earlier) as I don't feel competent enough as there are some things I've not seen

[GitHub] nifi issue #817: NIFI-2380 - Introduce ExtractTNEFAttachments

Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/817 @olegz ? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the

[GitHub] nifi issue #817: NIFI-2380 - Introduce ExtractTNEFAttachments

Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/817 @olegz I guess you have been busy but I was wondering if you are still planning to review this? Cheers --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] nifi issue #817: NIFI-2380 - Introduce ExtractTNEFAttachments

Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/817 @olegz how is testing looking like? Cheers --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this

[GitHub] nifi issue #817: NIFI-2380 - Introduce ExtractTNEFAttachments

Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/817 @olegz Rebased. Please note that while I will be happy to keep rebasing as peer review is in progress, this PR makes additions to the nifi-assembly LICENSE and NOTICE files so it is

[GitHub] nifi issue #817: NIFI-2380 - Introduce ExtractTNEFAttachments

Github user olegz commented on the issue: https://github.com/apache/nifi/pull/817 @trixpan Thank you and I will definitely take a look. Also, FWIW I see already that it has some merge conflicts so you may want to look and fix those. --- If your project is set up for it, you can

[GitHub] nifi issue #817: NIFI-2380 - Introduce ExtractTNEFAttachments

Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/817 @joewitt I will need some assistance with the L issue: Note how POIs LICENSE makes mention to CurvesAPI https://github.com/apache/poi/blob/trunk/legal/LICENSE#L459

[GitHub] nifi issue #817: NIFI-2380 - Introduce ExtractTNEFAttachments

Github user joewitt commented on the issue: https://github.com/apache/nifi/pull/817 Hello. The POI-Scratchpad dependency version being referenced as 'RELEASE' does not look correct. It appears the latest release version of poi-scratchpad is 3.14. Also, the PR needs to be updated

[GitHub] nifi issue #817: NIFI-2380 - Introduce ExtractTNEFAttachments

Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/817 @JPercivall - Given your review of ExtractEmailAttachments do you mind reviewing this PR? Cheers --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] nifi issue #817: NIFI-2380 - Introduce ExtractTNEFAttachments

Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/817 @joewitt - 3rd party test files have been replaced with new files containing only NiFi & MiNiFi logos --- If your project is set up for it, you can reply to this email and have your reply appear on