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

Minor fix for get_asset_url and documentation for same #161

Merged
merged 4 commits into from
Jan 3, 2020
Merged

Conversation

rpkyle
Copy link
Contributor

@rpkyle rpkyle commented Jan 3, 2020

This PR proposes to introduce a check that getAppPath() returns FALSE if the environment variable DASH_APP_ROOT_PATH is undefined when a Dash app was launched without using source(). In addition, documentation is added for the method.

Additionally, this PR contributes the url_base_pathname parameter, which existed in Dash for Python but not in Dash for R.

@HammadTheOne

Copy link
Contributor

@HammadTheOne HammadTheOne left a comment

Choose a reason for hiding this comment

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

Looks good to me, the documentation update is nice and the method works without issues with various tests of assets including those within nested directories.

💃

@rpkyle rpkyle merged commit fb05d18 into dev Jan 3, 2020
@rpkyle rpkyle deleted the fix-asset-url branch January 3, 2020 21:56
@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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants