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

admin-records: account for system owned records #2848

Merged

Conversation

fenekku
Copy link
Contributor

@fenekku fenekku commented Sep 17, 2024

account_for_system_owner

@fenekku
Copy link
Contributor Author

fenekku commented Sep 17, 2024

Check backend for entity resolver in long-term. for now, this is good enough.

@@ -73,8 +78,7 @@ class SearchResultItemComponent extends Component {
collapsing
className="word-break-all"
>
{/* TODO needs to be expanded */}
{result.parent.access.owned_by.user}
{(recordOwner && recordOwner.user) || "System"}
Copy link
Contributor

Choose a reason for hiding this comment

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

please leave a comment with a link to backend fix issue so that we replace it with the long term solution :)

@kpsherva
Copy link
Contributor

agreed to this temporary fix and later on providing the long term backend solution

@fenekku
Copy link
Contributor Author

fenekku commented Sep 18, 2024

Added reference. Merging.

@fenekku fenekku merged commit fd7428e into inveniosoftware:master Sep 18, 2024
4 checks passed
@fenekku fenekku deleted the admin_records_drafts_owner_may_be_null branch September 18, 2024 12:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants