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.

Sign up for GitHub

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

Jump to bottom

[UUX] Improve BPN-EDC configuration screen #994

Closed
4 tasks
ds-crehm opened this issue May 16, 2024 · 6 comments · Fixed by #1137
Closed
4 tasks

[UUX] Improve BPN-EDC configuration screen #994

ds-crehm opened this issue May 16, 2024 · 6 comments · Fixed by #1137
Assignees
@mkanal
Labels
frontend Frontend related Issues release-hot-fix pbi is relevant for current release and for hotfix path spillover temporary label for spill over uux UUX feedback and related topics

Comments

@ds-crehm
Copy link
Contributor

ds-crehm commented May 16, 2024 •
edited by ds-mmaul
Loading

Hints / Details

R24.05 UUX feedback:

-- blocking --
Admin View – BPN-EDC Konfiguration
Search option unclear. Search is on the top but actually searching in the section on the bottom. It would be much better to take out the "Add function" from the screen. Go via an "Add Connector" button and work with overlays or half screens .

  • Either make the search functionality clear or rework the screen with overlays/half screens.
    Second option needs a concept.
    --> move search bar to bottom above the configuration and hide it if no configs are available

-- blocking --
Admin View – BPN-EDC Konfiguration
In the View “New Entries” can edit URL and click on save changes – but there original entry is shown and will be acknowledged
proposal : make fields in “new entries” read only, then user has to click remove and retype in the configuration fields

  • Fix "Save changes" functionality or make table read-only.
    --> make fields in new entries read-only

-- blocking --
Admin View – BPN-EDC Konfiguration
Window shows “deleted” , “added” and “changed” prior to “save changes” OPTION
However there is no option to edit / change – is there? If not, please remove.
330980676-29118034-12cd-41b9-b3db-8a4a99c61621

--> There is, and the indicators do work:
image

Acceptance Criteria

  • Fix search
  • Fix "Save changes" process
  • Remove unused "Deleted", "Added", "Changed" indicators
  • (alternatively) Create concept for new BPN-EDC configuration process as a whole.
@ds-crehm ds-crehm added frontend Frontend related Issues DISCUSSION_NEEDED This ticket needs discussion within teammembers concept issues describing to work on a concept uux UUX feedback and related topics labels May 16, 2024
@ds-mwesener ds-mwesener mentioned this issue May 16, 2024
Closed
10 tasks
@ds-mmaul ds-mmaul self-assigned this May 17, 2024
@ds-mmaul
Copy link
Contributor

done with catenax-ng#1258

@ds-crehm
Copy link
Contributor Author

Tested on E2E: LGTM
Ready for review

@ds-crehm ds-crehm assigned mkanal and unassigned ds-mmaul May 22, 2024
@ds-lcapellino ds-lcapellino mentioned this issue May 22, 2024
Merged
2 tasks
@ds-crehm ds-crehm added release-hot-fix pbi is relevant for current release and for hotfix path and removed DISCUSSION_NEEDED This ticket needs discussion within teammembers concept issues describing to work on a concept labels May 27, 2024
@ds-crehm
Copy link
Contributor Author

Feedback from Julia: Remove indicators completely.

@ds-crehm ds-crehm reopened this May 28, 2024
@ds-crehm ds-crehm added the spillover temporary label for spill over label May 28, 2024
@ds-mmaul ds-mmaul assigned ds-mmaul and unassigned mkanal Jul 2, 2024
ds-mmaul added a commit that referenced this issue Jul 2, 2024
@ds-mmaul
chore(admin): #994 improve bpn edc configuration
ebff94a
ds-mmaul added a commit that referenced this issue Jul 2, 2024
@ds-mmaul
chore(admin): #994 improve bpn edc configuration
c6ee4c9
@github-actions github-actions bot mentioned this issue Jul 2, 2024
Merged
2 tasks
ds-mmaul added a commit that referenced this issue Jul 2, 2024
@ds-mmaul
chore(admin): #994 improve bpn edc configuration
4c34df6
ds-mwesener added a commit that referenced this issue Jul 2, 2024
@ds-mwesener
Merge pull request #1137 from eclipse-tractusx/chore/#994-improve-bpn…
2bcb61e 
…-edc-config-view

Chore/#994 improve bpn edc config view
@ds-crehm
Copy link
Contributor Author

ds-crehm commented Jul 2, 2024

Tested on E2E: LGTM - indicators removed
Ready for review

@ds-crehm ds-crehm reopened this Jul 2, 2024
@mkanal
Copy link
Contributor

mkanal commented Jul 9, 2024

LGFM PO acceptance in behalf of @jzbmw.

@mkanal mkanal closed this as completed Jul 9, 2024
@ds-crehm
Copy link
Contributor Author

ds-crehm commented Jul 9, 2024

@jjeroch according to the feedback in eclipse-tractusx/sig-release#611 (comment):

ITEM 9 – blocking - no change for user. Still there is no option to edit/change and hence the Icon and Counter do not make sense to the user.

we have removed the indicators for the BPN-EDC configuration. It looks like this now:
grafik

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mkanal mkanal

Labels
frontend Frontend related Issues release-hot-fix pbi is relevant for current release and for hotfix path spillover temporary label for spill over uux UUX feedback and related topics
Projects
Trace-X
Status: done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Chore/#994 improve bpn edc config view eclipse-tractusx/traceability-foss
3 participants
@mkanal @ds-mmaul @ds-crehm