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

Next-generation gen_resources #212

Merged
merged 9 commits into from
Jun 28, 2023
Merged

Next-generation gen_resources #212

merged 9 commits into from
Jun 28, 2023

Conversation

etpinard
Copy link
Collaborator

Mostly cherry-picking from #208, this PR updates the 18+ months old artifacts with resources from dash v2.10.2.

The updated tarball is hosted on: https://github.com/plotly/DashCoreResources/releases/tag/v2.10.2%2B0

We had to make two modifications to our tests to make them pass with the new resources:

  • commit 408138f adapts the Promise in clientside callback test as now JS Promises do work inside clientside callbacks thanks to dash#2009, released in dash v2.4.0
  • commit 5cc67c3 adapts the devtools prop check test suite (essential what this piece of a very large patch did in python) now that nested props are valid since dash#1763, released in dash v2.1.0

and we also had to update two percy baselines to account for CSS changes.

I hope the instructions added in ff135bd will make this process accessible to other Dash.jl contributors in the future.

etpinard added 9 commits June 28, 2023 14:31
as the recent versions of dash (> 1.21.0) have
the `dash_renderer` merged within the dash repo.

Namely, we need
- grab the `dash_renderer` module from the main `dash` module
- to override the `namespace` value
  with `"dash_renderer" to fetch from the
  correct URL, we do something similar for the renderer deps
  just above.
bump the version to use in `Sources.toml`
then commmit the updated `Artifacts.toml`
after `generate.jl --deploy` pointing to
https://github.com/plotly/DashCoreResources/releases/tag/v2.10.2%2B0
clientside callbacks with Promises are now supported
by dash since v2.4.0
`invalid-nested-prop` became `allow-nested-prop`
in dash 2.1.0

moreover, use same css query as in the python
version of the `test_props_check.py` suite
@github-actions github-actions bot added documentation Improvements or additions to documentation tests labels Jun 28, 2023
@etpinard etpinard added the priority Dash.jl tasks that should be completed prior to non-priority items label Jun 28, 2023
@etpinard
Copy link
Collaborator Author

I've been personally using the updated v2.10.2 artifacts for about a week now without issues.

So, I'll merge this branch into dev and let it sit there so that other Dash.jl developers can try the new artifacts out using

pkg> add Dash#dev

I'm planning on releasing the new artifacts as part of Dash.jl@v1.3.0 1-2 weeks from now, right before JuliaCon 2023.

@etpinard etpinard merged commit fece921 into dev Jun 28, 2023
@etpinard etpinard deleted the nextgen-gen_resources branch June 28, 2023 19:53
This was referenced Jul 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation priority Dash.jl tasks that should be completed prior to non-priority items tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant