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(organizations): Use owner_label for asset ownership display TASK-1181 #5327

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

rajpatel24
Copy link
Contributor

📣 Summary

Updated the frontend to read owner_label from the assets endpoint instead of owner__username for displaying the owner field in the asset list.

📖 Description

The owner field in the asset list UI now utilizes the owner_label field from the assets API endpoint.

The owner_label field in the Asset API dynamically displays:

  • The username of the asset owner if the organization is not a multi-member organization (MMO).
  • The name of the organization if the organization is a multi-member organization (MMO) and the user is the organization owner.

💭 Notes

  • Sorting and filtering are not yet updated to work with owner_label instead of owner__username.
  • No visual changes or additional functionality were introduced apart from reading and displaying the owner_label.

@rajpatel24 rajpatel24 changed the title feat(organizations): Read owner_label from the assets endpoint instead of owner__username TASK-1181 feat(organizations): Use owner_label for asset ownership display TASK-1181 Dec 4, 2024
@rajpatel24 rajpatel24 self-assigned this Dec 4, 2024
@noliveleger noliveleger requested review from RuthShryock and removed request for magicznyleszek and noliveleger December 4, 2024 20:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant