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

Bug Fix: Only override forceSvg from localStorage when the key is present #5834

Merged
merged 1 commit into from Aug 2, 2022

Conversation

rileyajones
Copy link
Contributor

  • Motivation for features / changes
    We always override the value of forceSvg regardless of whether a value is set in localStorage. This is an existing bug that is only coming to light now that we persist feature flags to the queryParams.
    Stop removing experiment related query parameters #5717

  • Detailed steps to verify changes work correctly (as executed by you)

  1. Clear localStorage
  2. Visit tensorboard and note that the forceSVG query parameter is not set.

Copy link
Contributor

@JamesHollyer JamesHollyer left a comment

Choose a reason for hiding this comment

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

LGTM

@rileyajones rileyajones merged commit 39d1f53 into tensorflow:master Aug 2, 2022
yatbear pushed a commit to yatbear/tensorboard that referenced this pull request Mar 27, 2023
…sent (tensorflow#5834)

only override forceSVG from localStorage when the key is present
dna2github pushed a commit to dna2fork/tensorboard that referenced this pull request May 1, 2023
…sent (tensorflow#5834)

only override forceSVG from localStorage when the key is present
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.

None yet

2 participants