Skip to content

add WQP status links - #925

Open
ehinman wants to merge 2 commits into
DOI-USGS:developfrom
ehinman:wqp-status-page-link
Open

add WQP status links#925
ehinman wants to merge 2 commits into
DOI-USGS:developfrom
ehinman:wqp-status-page-link

Conversation

@ehinman

@ehinman ehinman commented Aug 27, 2026

Copy link
Copy Markdown
Collaborator

This PR adds a description of the WQP status page and a link to the page from the function documentation and the status vignette. Brainstormed best places to put this link with Kiro and had it write an initial description, which I then edited.

Comment thread vignettes/Status.Rmd Outdated

```{r}
API_USGS_PAT = "[your api key]"
API_USGS_PAT <- "[your api key]"

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This doesn't work, it needs to be an =, the .Renviorn file:
https://rstats.wtf/r-startup.html#renviron
" This is not R code, it uses a format similar to that used on the command line shell."

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Ugh, this appears to be an "autocorrect" feature of positron for r chunks. Let me turn that off...

Comment thread R/readWQPdata.R
#' Sampling Activity \tab ActivityWQX3 \tab /wqx3/Activity/search \cr
#' }
#'
#' If you experience unexpected errors, check the

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Since you are changing the roxygen stuff, there should be an updated .Rd file pushed as well. THAT BEING SAID... CRAN gets really angry if urls provided in the help pages break or have a redirect. I'm a hesitant to add this link here - I don't know how often rconnect.usgs.gov goes down for maintenance or other reasons. Or, we could add the link here (so it shows up on the package documentation), but just remember to take it out on the submitted CRAN package (?)

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