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

Update: Metric label v2 #1062

Merged
merged 3 commits into from Oct 7, 2020
Merged

Conversation

kevinhinterlong
Copy link
Member

Resolves #1002

Description

Updates the metric label visualization for request v2, updates metadata to use cid

Proposed Changes

  • fixed an issue where visualizations would always appear to be valid for requests causing them to not be rebuilt
  • fixed a ts issue with not registering models
  • fixed an issue with accessing canonicalName in hasRequestRun where it got a serialized request instead of a request fragment

License

I confirm that this contribution is made under an MIT license and that I have the authority necessary to make this contribution on behalf of its copyright owner.

@kevinhinterlong kevinhinterlong added this to In progress in Request V2 via automation Oct 2, 2020
@kevinhinterlong kevinhinterlong linked an issue Oct 2, 2020 that may be closed by this pull request
5 tasks
@kevinhinterlong kevinhinterlong moved this from In progress to Review in progress in Request V2 Oct 2, 2020
@kevinhinterlong kevinhinterlong added the Visualization Features/issues related to data visualization label Oct 2, 2020
Request V2 automation moved this from Review in progress to Reviewer approved Oct 5, 2020
@kevinhinterlong kevinhinterlong merged commit e69b996 into request-v2 Oct 7, 2020
Request V2 automation moved this from Reviewer approved to Done Oct 7, 2020
@kevinhinterlong kevinhinterlong deleted the request-v2-metric-label branch October 7, 2020 19:19
jkusa pushed a commit that referenced this pull request Nov 2, 2020
* Update: Metric label v2
* Update: Docs fixup
* Update: Address pr comments
jkusa pushed a commit that referenced this pull request Nov 2, 2020
* Update: Metric label v2
* Update: Docs fixup
* Update: Address pr comments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Visualization Features/issues related to data visualization
Projects
No open projects
Request V2
  
Done
Development

Successfully merging this pull request may close these issues.

RequestV2: Update Label Visualization
4 participants