eclipse-tractusx / traceability-foss

https://eclipse-tractusx.github.io/traceability-foss/docs/
Apache License 2.0
10 stars 19 forks source link

[CONTRACT_MGMT][CONCEPT] "Contract Agreement Admin View" show assets for contractID's #737

Closed mkanal closed 11 hours ago

mkanal commented 3 months ago

As Trace-X admin I want to view assets for contract id so that could investigate the related assets for a contract id

Link

Hints / Details

Acceptance Criteria

Out of Scope

mkanal commented 2 weeks ago

@ds-crehm why have you added the BLOCKED label. Please add a reasonable info.

ds-crehm commented 1 week ago

Only a single contract can be selected to filter for.

Concept is here: https://github.com/catenax-ng/tx-traceability-foss/blob/main/docs/concept/%23737-contract-parts-link/%23737-contract-parts-link.md

Backend PBI: https://github.com/eclipse-tractusx/traceability-foss/issues/1036 Frontend PBI: https://github.com/eclipse-tractusx/traceability-foss/issues/1037

Ready for review

mkanal commented 1 week ago

Review

ds-crehm commented 1 day ago

It's already implemented: https://github.com/eclipse-tractusx/traceability-foss/issues/1037 Regarding your feedback:

  1. Understandable, however all other actions in Trace-X rely on the tooltips to convey the description of the action. So that's what I did here as well. We can create a ticket to change it, if you still want it differently.
  2. It would be handled just like the situation when a user selects multiple filters - no workaround needed
  3. In the contracts API, it is called "contractID". And this is the value we are filtering for. grafik In the frontend, there is no mention of Contract Agreement ID or Negotiation ID. grafik So I went with contractID and think this is the correct choice.
  4. The filter is always set, regardless of the visibility of the table. After the filters are set, the user can manage them individually for both tables.
  5. I'll discuss the error handling with @ds-mmaul and make sure that it is covered sufficiently.
mkanal commented 11 hours ago

PO acceptance in behalf of @jzbmw. LGFM all ac resolved.