MARP-1440 Finalize chat gpt assistant fix for explain feature in v11

[tvtphuc-axonivy]

No description provided.

[github-actions]

Test Results

8 tests  ±0   8 ✅ ±0   14s ⏱️ +2s
2 suites ±0   0 💤 ±0 
2 files   ±0   0 ❌ ±0 

Results for commit 07a983e. ± Comparison against base commit 86cd5ca.

♻️ This comment has been updated with latest results.

[ndkhanh-axonivy]

LGTM

[tvtphuc-axonivy]

Hi @ivy-rew,

Currently, I am working on story https://1ivy.atlassian.net/browse/MARP-1440. When I made a release for story https://1ivy.atlassian.net/browse/MARP-747 in version 10, I encountered a compile error that required 3 arguments instead of 2 in class openai-assistant/src/com/axonivy/connector/openai/assistant/DesignerClient.java .

I fixed this issue and updated the pom file to use "nightly-10.0" and running "mvn clean install".

It seems like it cannot find the library in version 10: javax.ws.rsch.ivyteam.ivy.rest.client.exec'.

I tried removing this from the MANIFEST.MF file. Now the build has passed, I tested it and it works well in version 10.

Do you agree that if I remove "javax.ws.rsch.ivyteam.ivy.rest.client.exec" in the MANIFEST.MF file for only version 10 ?

[ivy-rew]

Yes @tvtphuc-axonivy , for release/1o omitting the rest.client.exec bundle is a good solution. thanks for your efforts.

[ivy-rew]

However, please squash your commits before merging. Otherwise we get a lot of noise in the history for a simple build fix.

[tvtphuc-axonivy]

However, please squash your commits before merging. Otherwise we get a lot of noise in the history for a simple build fix.

Thanks Reguel, I will squash merge it.