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.

Sign up for GitHub

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

Jump to bottom

Modify R component prefixes to match Python, enforce dependency on dashR package #591

Closed
rpkyle opened this issue Feb 5, 2019 · 3 comments
Closed

Modify R component prefixes to match Python, enforce dependency on dashR package #591

rpkyle opened this issue Feb 5, 2019 · 3 comments
Assignees
@rpkyle

Comments

@rpkyle
Copy link
Contributor

rpkyle commented Feb 5, 2019 •
edited
Loading

For increased parity with the Dash for Python calling syntax, the current prefixes for dash-html-components and dash-core-components should be changed when generating R components:

  • html ➡️ dhc
  • core ➡️ dcc
@rpkyle rpkyle self-assigned this Feb 5, 2019
@alexcjohnson
Copy link
Collaborator

Our standard usage in Python is html for dash_html_components - so I think only core needs to change to dcc.

@rpkyle rpkyle changed the title Modify R component prefixes to match Python shorthand Modify R component prefixes to match Python, enforce dependency on dashR package Mar 8, 2019
@rpkyle
Copy link
Contributor Author

rpkyle commented Mar 8, 2019

The proposed implementation of the preload_dependency method, which will be useful for multipage DashR applications, requires that we enforce a dependency on DashR for all component library packages.

@rpkyle
Copy link
Contributor Author

rpkyle commented Mar 28, 2019

The above goals were achieved when #638 was merged into Dash. Closing this issue.

@rpkyle rpkyle closed this as completed Mar 28, 2019
HammadTheOne pushed a commit to HammadTheOne/dash that referenced this issue May 28, 2021
@rpkyle
Resolve loading issue related to plotly-1.4.9-min.js update in R (plo…
55128b3 
…tly#591)
HammadTheOne pushed a commit that referenced this issue Jul 23, 2021
@rpkyle
Resolve loading issue related to plotly-1.4.9-min.js update in R (#591)
70b338d
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rpkyle rpkyle

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@alexcjohnson @rpkyle and others