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 dash-renderer to 1.2.2 and fix dev tools UI display of stack traces #137

Merged
merged 6 commits into from
Dec 25, 2019

Conversation

rpkyle
Copy link
Contributor

@rpkyle rpkyle commented Nov 4, 2019

As noted in plotly/dash#994, the current version of dash-renderer bundled with Dash for R attempts to embed the R stack trace within an iframe.

This PR proposes to upgrade dash-renderer from v1.0.0 to v1.2.0, and also includes conditional logic to properly handle both Python and R stack traces, as described in plotly/dash#996.

Additional work may be required:

  • ensure dash_renderer.dev.js.map is up-to-date
  • ensure dash_renderer.min.js.map is up-to-date

@rpkyle rpkyle changed the title Update dash-renderer to 1.2.0 and fix dev tools UI display of stack traces Update dash-renderer to 1.2.2 and fix dev tools UI display of stack traces Dec 23, 2019
@rpkyle rpkyle requested review from shammamah-zz and removed request for alexcjohnson December 23, 2019 21:25
Copy link
Contributor

@shammamah-zz shammamah-zz left a comment

Choose a reason for hiding this comment

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

💃 As soon as tests pass!

@rpkyle rpkyle merged commit 93345e6 into dev Dec 25, 2019
@rpkyle rpkyle deleted the 994-fix-viewer branch December 25, 2019 22:12
@rpkyle rpkyle mentioned this pull request Jan 3, 2020
rpkyle added a commit that referenced this pull request Jan 4, 2020
* Provide support for no_update in Dash for R (#111)

* Use dev_tools_prune_errors instead of pruned_errors (#113)

* Better handling for user-defined error conditions in debug mode (#116)

* Provide support for multiple outputs (#119)

* Provide support for hot reloading in Dash for R (#127)

* Implement support for clientside callbacks in Dash for R (#130)

* Add line number context to stack traces when srcrefs are available (#133)

* Update dash-renderer to 1.2.2 and fix dev tools UI display of stack traces (#137)

* Support for meta tags in Dash for R (#142)

* Fixes for hot reloading interval handling and refreshing apps within viewer pane (#148)

* Support for asynchronous loading/compression in Dash for R (#157)

* Support returning asset URLs via public method within Dash class (#160)

* Minor fix for get_asset_url + docs, add url_base_pathname (#161)
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.

None yet

3 participants