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

feat: enhance technical user detail page #1093

Merged

Conversation

shahmargi12
Copy link
Contributor

@shahmargi12 shahmargi12 commented Sep 10, 2024

Description

  • Added the following fields to the technical user detail page:
    • User Type
    • Service Endpoint
  • Displayed a tooltip to handle missing information for older users.
  • Added translation for the tooltip text.

Note: Roles data, which is retrieved from the backend API under the key "roles," is already displayed on the page as "Permission."

Dependency:
This change depends on the backend update to include the usertype, authenticationServiceUrl fields in the API response from GET: api/administration/serviceaccount/owncompany/serviceaccounts/{GetServiceAccountId}. The backend implementation for this feature is tracked under sig#789 - API Expansion for Technical User Information portal-backend#976.

Why

To ensure complete service details are displayed and users are informed if information is missing for older profiles

Issue

1078

Checklist

  • I have followed the contributing guidelines
  • I have performed a self-review of my own code
  • I have successfully tested my changes locally

@MaximilianHauer MaximilianHauer added this to the Release 24.12 milestone Sep 13, 2024
Copy link
Contributor

@manojava-gk manojava-gk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lftm

Copy link

sonarcloud bot commented Sep 17, 2024

@oyo oyo merged commit fd9dca0 into eclipse-tractusx:main Sep 17, 2024
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: USER READY
Development

Successfully merging this pull request may close these issues.

4 participants