eclipse-tractusx / sig-release

https://eclipse-tractusx.github.io/sig-release
Apache License 2.0
9 stars 10 forks source link

Get data for Partner Network page through portal backend #958

Open ybidois opened 4 weeks ago

ybidois commented 4 weeks ago

Overview

Explain the topic in 2 sentences

We need to develop an API in portal-backend to get data from partner-network-pool instead of calling business-partner-pool service directly from the portal-frontend.

What's the benefit?

To have a consistency to call business-partner-pool services from backend side and this could also avoid firewall issues if we put business-partner-pool service is behind the firewall.

What are the Risks/Dependencies ?

Detailed explanation

We need to develop an API in portal-backend to get data from partner-network-pool instead of calling business-partner-pool service directly from the portal-frontend. This approach will make it consistent and secure, as we are already been calling business-partner-pool service from portal-backed to get legal-entity data by BPN.

API: https://business-partners-pool.example.com/v6/members/legal-entities/search?page=0&size=10

Current implementation

Currently frontend is been calling business-partner-pool directly to get Partner Network data.

Proposed improvements

Feature Team

Contributor

Committer

User Stories

Acceptance Criteria

Test Cases

Functionality and Test Cases should work as they are working right now because with this change, portal-frontend would be getting partner-network data from portal-backend instead of getting data directly from business-partner-pool and portal-backend would be calling business-partner-pool service internally and responding data to frontend.

Expected Result

  1. Frontend would be getting Partner Networks data from backend directly instead of calling business-partner-pool service directly.
  2. Backend API would be responding more or less the same data as it has been responded by business-partner-pool service.

Architectural Relevance

The following items are ensured (answer: yes) after this issue is implemented:

Justification: (Fill this out, if at least one of the checkboxes above cannot be ticked. Contact the Architecture Management Committee to get an approval for the justification)

Additional information

stephanbcbauer commented 1 week ago

Some hints from Release Management (@ther3sa) and Tractus-X Project Lead (@stephanbcbauer)

ybidois commented 1 week ago

In review based on comments from @ntruchsess and @oyo during refinement. May not be implemented in this form, depending on feedback from Cofinity-X infrastructure team.

stephanbcbauer commented 1 week ago

@evegufy mentioned ⇾ still under refinement