Skip to content

Conversation

byronz
Copy link
Contributor

@byronz byronz commented Sep 14, 2019

No description provided.

byron added 6 commits September 13, 2019 20:40
make percy assets configurable
add stop in case exception
add configure for percy assets path
📝 changelog
@byronz byronz marked this pull request as ready for review September 14, 2019 02:15
add devtool check in api, this will fail the test case, so we don't risk miss that from some weird percy auto merge
self.percy_snapshot(resource_path)
assert not self.driver.find_elements_by_css_selector(
"div.dash-debug-alert"
), "devtools should not raise an error alert"
Copy link
Collaborator

Choose a reason for hiding this comment

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

🏆

Copy link
Collaborator

Choose a reason for hiding this comment

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

though, might need an option to disable this in case someone wants to snapshot the errors

Copy link
Contributor Author

@byronz byronz Sep 14, 2019

Choose a reason for hiding this comment

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

Screen Shot 2019-09-13 at 11 54 45 PM
🎉 works like a charm

byron added 2 commits September 13, 2019 23:28
Copy link
Collaborator

@alexcjohnson alexcjohnson left a comment

Choose a reason for hiding this comment

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

Your call if slash-flexibility #923 (comment) is important. Aside from that question this looks great. 💃

Make sure to go back and remove the ref to this branch from dash-docs after this is released.

@byronz byronz merged commit e4ea7a4 into dev Sep 14, 2019
@byronz byronz deleted the percy-runner branch September 14, 2019 21:58
HammadTheOne pushed a commit to HammadTheOne/dash that referenced this pull request May 28, 2021
…down-#808

sets autoComplete to off in Dropdown plotly#808
HammadTheOne pushed a commit that referenced this pull request Jul 23, 2021
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.

2 participants