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

description field contains >1 description #561

Open
jordanpadams opened this issue Oct 30, 2024 · 1 comment
Open

description field contains >1 description #561

jordanpadams opened this issue Oct 30, 2024 · 1 comment
Labels
bug Something isn't working icebox s.low Low level severity

Comments

@jordanpadams
Copy link
Member

Checked for duplicates

Yes - I've already checked

🐛 Describe the bug

When I did https://pds.mcp.nasa.gov/api/search/1/products/urn:nasa:pds:context:instrument:mars2020.rover, I noticed the descriptions contain both the Instrument.description and the Modification_Detail.description

🕵️ Expected behavior

I expected the description field to make sense

📜 To Reproduce

  1. go to https://pds.mcp.nasa.gov/api/search/1/products/urn:nasa:pds:context:instrument:mars2020.rover
  2. note description property containing 3 descriptions

🖥 Environment Info

No response

📚 Version of Software Used

No response

🩺 Test Data / Additional context

No response

🦄 Related requirements

No response

⚙️ Engineering Details

We should discuss the actual use of this property. Is it intended to be a catchall of all descriptions in the label? Is this actually useful for the end user? Or is this really just intended for the eventual keyword search?

🎉 Integration & Test

No response

@jordanpadams jordanpadams added bug Something isn't working needs:triage labels Oct 30, 2024
@jordanpadams jordanpadams self-assigned this Oct 30, 2024
@jordanpadams jordanpadams added icebox s.low Low level severity s.medium Medium level severity and removed needs:triage s.medium Medium level severity labels Oct 30, 2024
@jordanpadams
Copy link
Member Author

Given low severity due to known workarounds elsewhere in the metadata

@jordanpadams jordanpadams removed their assignment Oct 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working icebox s.low Low level severity
Projects
Status: ToDo
Development

No branches or pull requests

1 participant