QuickBooks-V3-Java-SDK icon indicating copy to clipboard operation
QuickBooks-V3-Java-SDK copied to clipboard

Split package com.intuit.ipp.core between java-devkit and java-data

Open plcarmel opened this issue 5 years ago • 11 comments

It makes it impossible to include both of those maven dependencies directly (i.e.e without some form of repackaging) in a Java 9 modules.

plcarmel avatar Feb 13 '20 21:02 plcarmel

hey @plcarmel - I have added a story in our backlog to address this. Here's the internal jira id IDG-3475 for reference. We'll try to scope this work in our upcoming sprints. If you would like to contribute a fix for this, please send across a PR.

diana-derose avatar Feb 18 '20 18:02 diana-derose

Hi @fjolin-intuit, I would like to work on this

rachanamamillapalli avatar Sep 29 '20 05:09 rachanamamillapalli

@diana-derose / @plcarmel Is this still needed or some work is already going one?

rachanamamillapalli avatar Sep 29 '20 05:09 rachanamamillapalli

@rachanamamillapalli, that would be appreciated.

plcarmel avatar Sep 29 '20 11:09 plcarmel

@rachanamamillapalli This is still needed - your contribution is welcomed :)

fjolin-intuit avatar Sep 29 '20 16:09 fjolin-intuit

@plcarmel @rachanamamillapalli is this being looked upon? I wanted to understand the issue and take a stab onto fixing this over the weekend.

ashking94 avatar Oct 09 '20 13:10 ashking94

@ashking94, not by me.

plcarmel avatar Oct 09 '20 13:10 plcarmel

@fjolin-intuit @plcarmel @diana-derose Could you take a look at the PR linked to this issue.

rachanamamillapalli avatar Oct 17 '20 22:10 rachanamamillapalli

@rachanamamillapalli I spent some time looking at this - it would introduce some breaking changes which is ok as long as we version the release and put communication out beforehand. Were you able to test this PR against a sample? I tried this and couldn't get the dependencies to work out - I think it might be because the all project that was introduce is a generator pom instead of creating jars. I wasn't able to figure out how to import ipp-v3-java-all into my project. If you could try this in a sample to show how converting is done - we could create better communication around this change and I could bump up a major version change.

fjolin-intuit avatar Nov 09 '20 17:11 fjolin-intuit

@fjolin-intuit I haven't tested it on sample yet. Will try importing and update once I make any progress.

rachanamamillapalli avatar Nov 11 '20 09:11 rachanamamillapalli

Is there any workaround for this problem? module XXX reads package com.intuit.ipp.core from both ipp.v3.java.devkit and ipp.v3.java.data I cannot seem to find an answer anywhere. There must be some developers who have moved beyond Java 8 and circumvented this somehow.

othenos avatar Aug 30 '21 18:08 othenos