Conversation
There was a problem hiding this comment.
Pull request overview
Updates the Ocean Current products configuration to relocate SST climatology-related products under the existing sixDaySst product group, aligning the OC API product hierarchy with the intended grouping.
Changes:
- Added SST climatology and climatology data count entries under
sixDaySst. - Removed the standalone top-level
Climatologyproduct group.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| id: "sixDaySst-centiles" | ||
| - title: "Timeseries" | ||
| id: "sixDaySst-timeseries" | ||
| - title: "Climatology" |
There was a problem hiding this comment.
The new product title "Climatology" is ambiguous now that there is also a separate climatology data-count entry directly below. Since the id is "sixDaySst-climatologySst", consider making the title explicit (e.g., "SST Climatology" / "Climatology SST") so consumers/UI can distinguish it clearly.
| - title: "Climatology" | |
| - title: "SST Climatology" |
| id: "sixDaySst-timeseries" | ||
| - title: "Climatology" | ||
| id: "sixDaySst-climatologySst" | ||
| - title: "SSTAARS Data Count Data Count" |
There was a problem hiding this comment.
The title "SSTAARS Data Count Data Count" contains a duplicated "Data Count", which will show up as-is in any product listing. Please correct the label (e.g., remove the duplication, and/or simplify to "Data Count" if the context already implies SSTAARS).
| - title: "SSTAARS Data Count Data Count" | |
| - title: "SSTAARS Data Count" |
weited
deleted the
feature/8285-move-climatology-sst-products-under-sixdaysst-in-oc-api
branch
No description provided.