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

Updates for docstring links #134

Open

akeeste opened this issue May 14, 2024 · 1 comment

Labels

akeeste commented May 14, 2024

While combing through our documentation and API, I noticed a few links that should be updated, specifically in mhkit.tidal.io.noaa.request_noaa_data.m

https://tidesandcurrents.noaa.gov/api/ redirects to https://api.tidesandcurrents.noaa.gov/api/prod/. let's update it to use URL it ultimately redirects to
https://co-ops.nos.noaa.gov/api/ returns a privacy warning and ultimately redirects to https://api.tidesandcurrents.noaa.gov/api/prod/. let's update it to use URL it ultimately redirects to

akeeste added the documentation label

simmsa added a commit to simmsa/MHKiT-MATLAB that referenced this issue


          Fix: Issue MHKiT-Software#134, NOAA documentation links

303c9e3

`https://tidesandcurrents.noaa.gov/api/` to `https://api.tidesandcurrents.noaa.gov/api/prod/`
`https://co-ops.nos.noaa.gov/api/` to `https://api.tidesandcurrents.noaa.gov/api/prod/`

Contributor

simmsa commented Feb 17, 2025

@akeeste, thanks for reporting this. This was fixed here simmsa@303c9e3 and will be included in the upcoming v0.6.0 release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment