Skip to content
This repository was archived by the owner on Aug 29, 2025. It is now read-only.

Generate python components classes#243

Merged
T4rk1n merged 8 commits into
masterfrom
generate-class
Nov 15, 2018
Merged

Generate python components classes#243
T4rk1n merged 8 commits into
masterfrom
generate-class

Conversation

@T4rk1n

@T4rk1n T4rk1n commented Nov 15, 2018

Copy link
Copy Markdown
Contributor

Change the python component loading to generated classes files.

One thing to note, build:py now requires dash to be installed on the running python environment.

@chriddyp
chriddyp temporarily deployed to dash-table-review-pr-243 November 15, 2018 18:14 Inactive
@T4rk1n T4rk1n changed the title Generate class Generate python components classes Nov 15, 2018

@Marc-Andre-Rivet Marc-Andre-Rivet left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@T4rk1n This looks fine as far as I can tell.

Could you explain what the component classes are used for? Is this metadata for the IDE?

Otherwise, just need you to update the Changelog and to bump the version to make this PR complete.

@T4rk1n

T4rk1n commented Nov 15, 2018

Copy link
Copy Markdown
Contributor Author

@Marc-Andre-Rivet It's for IDE support, they can read the file instead of being generated at runtime.

@chriddyp
chriddyp temporarily deployed to dash-table-review-pr-243 November 15, 2018 20:04 Inactive
@chriddyp
chriddyp temporarily deployed to dash-table-review-pr-243 November 15, 2018 21:11 Inactive
@chriddyp
chriddyp temporarily deployed to dash-table-review-pr-243 November 15, 2018 21:27 Inactive
Comment thread .config/webpack/base.js
rules: [
{
test: /demo[/]index.html?$/,
test: /demo[/\\]index.html?$/,

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Oh.. windows!

@Marc-Andre-Rivet Marc-Andre-Rivet left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants