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

Better defaults for contact info panel #684

Merged
merged 3 commits into from
Aug 20, 2024
Merged

Conversation

MatMoore
Copy link
Contributor

@MatMoore MatMoore commented Aug 19, 2024

Part of #601

In order to present a consistent contact info panel, we should always show each of the three metadata fields, falling back to a "Not provided" value if missing.

If access information or contact information is missing, direct the user to the data owner. When this happens we would also like the data owner to add in the missing metadata.

If the data owner is missing, then direct the user to us. We do not intend to show any metadata without a data owner, so this should be treated as a bug in the service.

Examples

All metadata missing

Screenshot 2024-08-19 at 13 33 00

Access information missing

Screenshot 2024-08-19 at 13 32 31

Contact channels missing

Screenshot 2024-08-19 at 13 32 12

- if there is a data owner, direct users to them
- if not, this is probably a bug with ingestion. show "Not provided" in
  this case

hypothesis - this will get missing metadata issues fixed more quickly
In order to present a consistent contact info panel, we should always
show each of the three metadata fields, falling back to a "Not provided"
value if missing.

If access information or contact information is missing, direct the user
to the data owner. When this happens we would also like the data
owner to add in the missing metadata.

If the data owner is missing, then direct the user to us. We do not
intend to show any metadata without a data owner, so this should
be treated as a bug in the service.

part of #601
@MatMoore MatMoore marked this pull request as ready for review August 19, 2024 11:07
@MatMoore MatMoore requested a review from a team as a code owner August 19, 2024 11:07
Copy link
Collaborator

@mitchdawson1982 mitchdawson1982 left a comment

Choose a reason for hiding this comment

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

🚀

@MatMoore MatMoore merged commit 71fbb75 into main Aug 20, 2024
6 of 7 checks passed
@MatMoore MatMoore deleted the fmj-601-better-defaults branch August 20, 2024 14:25
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