Skip to content

Conversation

@willbarrett
Copy link
Member

CATEGORY

Choose one

  • Bug Fix
  • Enhancement (new features, refinement)
  • Refactor
  • Add tests
  • Build / Development Environment
  • Documentation

SUMMARY

#9522 was merged after we finally got CI to pass, but it contained an experimental fix recommended by Max that didn't actually resolve the issue. This PR reverts to the original fix, which does resolve the issue.

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TEST PLAN

ADDITIONAL INFORMATION

  • Has associated issue:
  • Changes UI
  • Requires DB Migration.
  • Confirm DB Migration upgrade and downgrade tested.
  • Introduces new feature or API
  • Removes existing feature or API

REVIEWERS

@villebro

@willbarrett willbarrett changed the title Viz py fix try 4 Viz py fix try 4 (Multi-line chart bug) Apr 14, 2020
@villebro
Copy link
Member

Oh, that's curious, I thought Max's solution would have fixed it, too. Let me think this through and get back in a second.

@villebro
Copy link
Member

@willbarrett
Copy link
Member Author

🤦 updating

@willbarrett
Copy link
Member Author

I'm actually going to wait and see if we actually get a clean build on this branch. No point in doing the update here if it Travis messes up again.

@willbarrett
Copy link
Member Author

Something is weird with history on this branch, closing.

@willbarrett willbarrett deleted the wbarrett/viz-py-fix-try-4 branch April 14, 2020 17:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants