Skip to content

ERM 3654: Replace HttpBuilderNg with Micronaut Http Client#902

Draft
mchaib-k-int wants to merge 10 commits intomasterfrom
ERM-3654
Draft

ERM 3654: Replace HttpBuilderNg with Micronaut Http Client#902
mchaib-k-int wants to merge 10 commits intomasterfrom
ERM-3654

Conversation

@mchaib-k-int
Copy link
Copy Markdown
Contributor

No description provided.

@github-actions
Copy link
Copy Markdown

License CLA Stuck? (Developer should make sure that it is really stuck before clicking)

@github-actions
Copy link
Copy Markdown

github-actions bot commented Jun 18, 2025

Test Results

 27 files  ±0   27 suites  ±0   10m 39s ⏱️ +2s
647 tests ±0  643 ✅ ±0  4 💤 ±0  0 ❌ ±0 
648 runs  ±0  644 ✅ ±0  4 💤 ±0  0 ❌ ±0 

Results for commit bf6d8b3. ± Comparison against base commit 5a39434.

♻️ This comment has been updated with latest results.

@EthanFreestone
Copy link
Copy Markdown
Contributor

Things for discussion here... namely these shouldn't all be relying on GokbClient and I think there's still need for a webSourceAdapter... We can potentially handle this with an "AdapterClient" interface or otherwise and then have the websource adapter code utilise one of those.... Alternatively if we're going separate per adapter (which might be the right model... I haven't thought about this totally yet) then my gut instinct says that we need to set up dummy clients for the other adapters, they can just throw "Is not implemented" for now though since those adapters aren't used....

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

Successfully merging this pull request may close these issues.

2 participants