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

Remove TWS Dataset #412

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

Remove TWS Dataset #412

wants to merge 3 commits into from

Conversation

siddharth0248
Copy link
Contributor

Why are you creating this Pull Request?

Request from Wanshu (EIS team):
could you help remove the following two datasets. Many thanks!

TWS anomalies: https://www.earthdata.nasa.gov/dashboard/data-catalog/tws-anomaly
TWS trend: https://www.earthdata.nasa.gov/dashboard/data-catalog/tws-trend

Copy link

netlify bot commented Jun 25, 2024

Deploy Preview for visex ready!

Name Link
🔨 Latest commit 4e1a625
🔍 Latest deploy log https://app.netlify.com/sites/visex/deploys/667c4a4b07457c0008190087
😎 Deploy Preview https://deploy-preview-412--visex.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@siddharth0248 siddharth0248 requested review from slesaad June 25, 2024 16:55
@slesaad
Copy link
Member

slesaad commented Jun 25, 2024

@siddharth0248 i see at least one story that references one of the datasets (there may be more references elsewhere too). the story may need to be updated based on the removed dataset?

also, do they wanna remove it from the stac catalog and the data store too? if so, that'll be another ticket.

Copy link
Member

@slesaad slesaad left a comment

Choose a reason for hiding this comment

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

need to update references in the stories

maybe just hide the datasets?

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