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

Show default display name on analyses #4924

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from
Open

Show default display name on analyses #4924

wants to merge 2 commits into from
+15 −14

Conversation

perryr16
Copy link
Contributor

@perryr16 perryr16 commented Jan 14, 2025 •
edited
Loading

Any background context you want to provide?

What's this PR do?

Shows property default display field in the analyses table

How should this be manually tested?

  • create an analysis
  • property's default display field should be in the analyses table. If the property is missing the default display field, it will display "Property: Unknown"

What are the relevant tickets?

#4822

Screenshots (if appropriate)

Screenshot 2025-01-14 at 2 14 28 PM

@perryr16 perryr16 added the Bug label Jan 14, 2025
@perryr16 perryr16 marked this pull request as ready for review January 14, 2025 21:06
@perryr16 perryr16 requested a review from kflemin January 14, 2025 21:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@haneslinger haneslinger haneslinger approved these changes

@kflemin kflemin Awaiting requested review from kflemin

Assignees
No one assigned
Labels
Bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@perryr16 @haneslinger @kflemin