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

patch timeseries identifier needs to be updated #854

Open
Enovotny opened this issue Aug 20, 2024 · 2 comments
Open

patch timeseries identifier needs to be updated #854

Enovotny opened this issue Aug 20, 2024 · 2 comments
Labels
approved-W192HQ23F0232-task4 Only valid if set by MikeNeilson, DanielO, CharlesG

Comments

@Enovotny
Copy link

the patch timeseries identifier endpoint required two timeseries ids both labeled timeseries-id. I don't know if this is just an error on the swagger page or internally. still using version 1 so I am guessing it just hasn't been updated.

image

@adamkorynta
Copy link
Collaborator

One is the path parameter and the other the query parameter. The path parameter should be updated to {name} to follow standards.

@MikeNeilson MikeNeilson added the approved-W192HQ23F0232-task4 Only valid if set by MikeNeilson, DanielO, CharlesG label Sep 3, 2024
@Enovotny
Copy link
Author

Enovotny commented Sep 3, 2024

should probably be name and new-timeseries-id or new-name. So you know that is what the timeseries-id/name is going to be updated to.

adamkorynta pushed a commit that referenced this issue Oct 8, 2024
Issue [#854](#854) - Rename
TS Identifier patch path parameter
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved-W192HQ23F0232-task4 Only valid if set by MikeNeilson, DanielO, CharlesG
Projects
None yet
Development

No branches or pull requests

3 participants