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.

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

Feedback: Identify that targetTime is period ending in swagger #177

Open
TheLiamGuy opened this issue Feb 24, 2023 · 1 comment
Open

Feedback: Identify that targetTime is period ending in swagger #177

TheLiamGuy opened this issue Feb 24, 2023 · 1 comment
Labels
good first issue Good for newcomers

Comments

@TheLiamGuy
Copy link

TheLiamGuy commented Feb 24, 2023 •
edited
Loading

Currently, the description text for targetTime says "The target time that the forecast is produced for"

To help interpretation, would be helpful to add. "Format is period-ending (e.g. a time ending in 12:30 refers to the period 12-12:30)"

Proposed full text: "The target time for which the forecast is produced. Format is period-ending (e.g. a time ending in 12:30 refers to the period 12-12:30)."

Discussed with @peterdudfield
@TheLiamGuy TheLiamGuy added the enhancement New feature or request label Feb 24, 2023
@peterdudfield
Copy link
Collaborator

Thanks @TheLiamGuy

@peterdudfield peterdudfield added good first issue Good for newcomers and removed enhancement New feature or request labels Feb 28, 2023
@peterdudfield peterdudfield changed the title Identify that targetTime is period ending in swagger Feedback: Identify that targetTime is period ending in swagger Mar 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
good first issue Good for newcomers
Projects
No open projects
Nowcasting WP4
Status: No status
Milestone
No milestone
Development

No branches or pull requests
2 participants
@TheLiamGuy @peterdudfield