Skip to content
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

Migrate Catalog Browser to API Wrapper #232

Closed
13 tasks
SebastianOpriel opened this issue Apr 19, 2023 · 1 comment
Closed
13 tasks

Migrate Catalog Browser to API Wrapper #232

SebastianOpriel opened this issue Apr 19, 2023 · 1 comment
Assignees
Labels
kind/enhancement New feature or request
Milestone

Comments

@SebastianOpriel
Copy link
Member

SebastianOpriel commented Apr 19, 2023

Feature

Description

Right now the catalog browser does multiple requests in the UI and cannot properly include further information for contract offers such as existing negotiations or existing contract agreements

Possible Solution

  • In general, this is on hold until our broker server / broker client have been implemented further so we can judge how the broker and the new catalog browser will interact.

MVP

  • Create API Wrapper Endpoint (MVP endpoint, only include asset for each contract offer)
    • Accepts a list of additional Connector Endpoints
    • Has a flag to skip preconfigured Connector Endpoints
    • Requests Connector Endpoints in parallel
    • Returns a status for the entire request, in case all requests failed.
    • Returns a list of request status results, e.g. for each url if it succeeded, and if it didn't, a comprehensible user error message
    • Returns a merged list of contract offer dtos.
    • Accept search parameters, search should happen on the backend side.
  • Migrate the UI to use the new endpoint. This should allow us to remove existing logic about merging results / merging status results

Potential Improvements (to be implemented later)

@richardtreier
Copy link
Collaborator

replaced by #403

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants