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 highlight.js dependency for dash-table #262

Merged
merged 3 commits into from
Aug 11, 2021
Merged

Update highlight.js dependency for dash-table #262

merged 3 commits into from
Aug 11, 2021

Conversation

HammadTheOne
Copy link
Contributor

@HammadTheOne HammadTheOne commented Jun 21, 2021

This PR updates the component libraries for Dash R and fixes the following issue with the async-highlight.js dependency sourcing when a dash-table is present within the layout:

image

This issue is caused by this shared dependency between dash-core-components and dash-table. When we merge the dependency artifacts into inst/deps, the newer version of the dependency sourced from dash-table is overwritten by the file sourced from dash-core-components. This PR adds a change so that only the more recent version from dash-table is sourced for this dependency. As a longer-term solution we may want to sync up these dependencies between the two repos.

@alexcjohnson
Copy link
Collaborator

After this change, does dccMarkdown work or does it yield a similar error? I'm worried that we may need to serve both files, ie put one of them in a different place or give it a different name, and adjust .dashCoreComponents_js_metadata or .dashTable_js_metadata accordingly.

@HammadTheOne
Copy link
Contributor Author

HammadTheOne commented Jun 24, 2021

After this change, does dccMarkdown work or does it yield a similar error? I'm worried that we may need to serve both files, ie put one of them in a different place or give it a different name, and adjust .dashCoreComponents_js_metadata or .dashTable_js_metadata accordingly.

I didn't notice any issues with dccMarkdown when using the different version of highlight.js. Testing out the examples in the docs, there's no discernible difference between the two. I think if this has the possibility of happening with other dependencies then it might be worth exploring other ways to serve the files - we could even put dependencies into separate directories (eg. dcc, html, table) like we're doing with the Python monorepo, but that would require some larger changes.

@alexcjohnson
Copy link
Collaborator

I don't understand the asymmetry - aside from a slight version difference in the underlying libraries (10.3.1 vs 10.6.0) the two chunks look identical. The dash-table lazy loader and its target are exact copies of the DCC lazy loader and its target. So why would DCC work with the table chunk if the table doesn't work with the dcc chunk?

The DCC built chunk starts with:

(window.webpackJsonpdash_core_components=window.webpackJsonpdash_core_components||[]).push(

whereas the table chunk starts with:

(window.webpackJsonpdash_table=window.webpackJsonpdash_table||[]).push(

Which are pretty clearly incompatible with each other. So please check it out again - maybe your browser had a cached version of the DCC highlight chunk? I really think we're going to need to serve both, but we should be able to do so without restructuring the entire deps folder, just making a special case for this file.

You're right though that long-term we want to DRY this - see my comment https://github.com/plotly/dash-core/issues/323#issuecomment-867651335 for one possibility.

@HammadTheOne
Copy link
Contributor Author

HammadTheOne commented Jun 25, 2021

You were right about the browser having cached the dcc version of the highlight chunk. After clearing cache and relaunching the app, it fails to load a different chunk required by dccMarkdown (the component renders, but I'm guessing there will be problems with syntax highlighting).

I think merging DataTable into dcc makes sense, but in the interim, I've changed the file names to core-async-highlight.js and table-async.highlight.js respectively (along with the sourcemaps), updated the JS metadata, and tested that it's working without any other console errors.

@alexcjohnson
Copy link
Collaborator

I've changed the file names to core-async-highlight.js and table-async.highlight.js respectively (along with the sourcemaps), updated the JS metadata, and tested that it's working without any other console errors.

@HammadTheOne are those changes ready to push? I think this PR will be ready at that point.

@HammadTheOne
Copy link
Contributor Author

@alexcjohnson Yep, sorry about that, got a little sidetracked. Changes have been pushed in 86105c7.

Copy link
Collaborator

@alexcjohnson alexcjohnson left a comment

Choose a reason for hiding this comment

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

💃

@HammadTheOne HammadTheOne merged commit a055015 into dev Aug 11, 2021
@HammadTheOne HammadTheOne deleted the fix-table branch August 11, 2021 14:57
HammadTheOne added a commit that referenced this pull request Oct 13, 2021
* contribute test script

* remove version updating in DESCRIPTION

* fix EOL

Co-authored-by: HammadTheOne <[email protected]>

* Add support for user-defined server routes (#225)

* Provide support for script and stylesheet attributes (#226)

* Authenticate on pulls from Docker Hub (#231)

* Add support for callback graph improvements and timing (#224)

* Update CHANGELOG.md

* 189 - Add Pattern Matching Callbacks for Dash R (#228)

* Testing initial implementation

* More testing

* Callback Context Updates

* Updating callback context logic

* Fixing callback returns

* Adding callback args conditional

* Cleanup and additional changes to callback value conditionals

* Comment cleanup

* Added PMC callback validation, removed unnecessary code

* Update R/dependencies.R

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Update R/dependencies.R

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Update R/dependencies.R

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Update R/dependencies.R

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Added build to gitignore

* Updated dependencies.R

* Update boilerplate docs and add wildcard symbols

* Drying up validation code and applying symbol logic

* Update test to use symbols

* Cleaned up code and added allsmaller test example

* Cleaning up redundant code

* Update FUNDING.yml

* Updated callback_args logic and example

* Adding basic unittests, updated validation

* Fixed response for MATCH callbacks

* Added integration test and updated examples for docs

* Added additional integration test

* Formatting and cleanup

* update docs

* Update to-do app

* Add comments to examples

* Change empy vector to character type.

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Update boilerplate text.

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Update tests/integration/callbacks/test_pattern_matching.py

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Update tests/integration/callbacks/test_pattern_matching.py

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Update tests/integration/callbacks/test_pattern_matching.py

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Update tests/integration/callbacks/test_pattern_matching.py

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Update tests/integration/callbacks/test_pattern_matching.py

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Update tests/testthat/test-wildcards.R

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Update wildcards_test.R

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Update wildcards_test.R

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Update wildcards_test.R

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Update wildcards_test.R

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Update wildcards_test.R

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Update wildcards_test.R

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Update wildcards_test.R

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Update wildcards_test.R

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Removed triple colon syntax

* Use seq_along and remove unnecessary unittest

* Update CHANGELOG.md

* Update CHANGELOG.md

* Add support for arbitrary and sorted keys

* Whitespace deleted

* Added integration tests

* Fixing test output

* Fixing flakiness

* Update test_pattern_matching.py

* Update test_pattern_matching.py

* Updating boilerplate text and test with generalized keys

* Minor test fixes

Co-authored-by: Ryan Patrick Kyle <[email protected]>
Co-authored-by: Nicolas Kruchten <[email protected]>
Co-authored-by: rpkyle <[email protected]>

* Fixing Null error with glue::glue interpolation (#233)

* Fixing NULL error with glue interpolation

* Update utils.R

* Update utils.R

* Update CHANGELOG.md

* Update dash-renderer to v1.8.2 (#234)

* bump dash-renderer to v1.8.2

* Update CHANGELOG.md

* add note about update to dash-renderer

* Fixing flaky test

* bump package version to v0.8.0

* Update R/dash.R

Co-authored-by: HammadTheOne <[email protected]>

* Update tests/testthat/test-wildcards.R

Co-authored-by: HammadTheOne <[email protected]>

* Update DESCRIPTION

Co-authored-by: HammadTheOne <[email protected]>

* add PMC example

* update documentation

* update CHANGELOG release date

* 🔨 PMC docs refactor

* Update tests/integration/callbacks/test_pattern_matching.py

Co-authored-by: HammadTheOne <[email protected]>

* Update tests/integration/callbacks/test_pattern_matching.py

Co-authored-by: HammadTheOne <[email protected]>

* Update tests/integration/callbacks/test_pattern_matching.py

Co-authored-by: HammadTheOne <[email protected]>

* add import of glue

* add glue to imports.R

* fix line length issue

* Fix setCallbackContext for wildcard and ordinary inputs (#237)

* Update setCallbackContext

* Adding graphs test

* Slight fix

* bump version and update CHANGELOG

* Less flaky test

Co-authored-by: rpkyle <[email protected]>

* bump dependency versions

* update CHANGELOG

* update dash-renderer to v1.8.3

* update CHANGELOG

* Favicon fix (#240)

* Adding default favicon

* Removing redundant codeblock

* Added default favicon

* Minor fix to requests prefix

* Update CHANGELOG.md

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Added simple test

* Fixed typo

* Fixed typo

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Remove context reference from CircleCI (#241)

* Dash R Core Package Unification (#243)

* Initialize npm and gulpfile in repo

* Adding directory structure

* Initial implementation of unification script

* Minor grep fixes

* Fixed DESCRIPTION imports

* Added updated dash-table deps

* Regex for version numbers

* Cut import entries from NAMESPACE

* Remove && include(dashr)

* Removing gulp-asset artifacts and rebuilding complete package

* Removing unnecessary files

* fix: remove html, core pkgs from tests

* fix: update script tags unit test

* Revert R6 import

* Add temporary collate

* Update README examples

* Scrubbing imports

* More import scrubbing

* Package development updates

* Update gitignore and namespace

* Updated gulpfile jobs

* Updated all dependencies

* Added templates for namespace/internal exports

* Update internal, namespace, and gulpfile cleanup

* Fix dependency sourcing

* Linting

* Adding job for asset retrieval and deletion

* Minor src change

* Added error handling

* Fixing favicon bug

* chore: use shallow clone

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Added deprecation warning if dcc, html, or table packages are attached (#249)

* Added deprecation warning

* Update R/dash.R

Co-authored-by: Ryan Patrick Kyle <[email protected]>

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Update `highlight.js` dependency for dash-table (#262)

* Updating gulpfile

* Updating dependencies for dash components

* Updated highlight.js dependency source

* Add Dash 2 layout syntax wrappers and html tags (#265)

* Added add_meta helper

* Add helper functions and export pipe

* Adding in tags wrapper

* Updating ci config

* Updated CHANGELOG

* Updated circleci to include rust package manager (cargo).

* Removed dashr command in circleci config.

* Updating changelog

* added basic test

* Cleaning up Dash 2 references and duplication

* Updated documentation

* Allow conditional UI

* Add meta tag check

* More cleanup

* Last bit of cleanup

Co-authored-by: Steve Sperandeo <[email protected]>

* Fix suppress_callback_exceptions config (#268)

* Add config key

* Fixing CI

* Simplified callback syntax and addtional utility functions (#270)

* Tag updates

* Added RStudio dash snippet

* Added simple_table

* Added flexible callbacks

* Documentation and NAMESPACE updates

* Updated DESCRIPTION

* Adding unittests

* Adding context tags to tests

* Updated CHANGELOG

* Update monorepo and rebuild package (#271)

* Updating gulpfile and package.json

* More package.json and linting updates

* Adding in simplified callback updates/tests

* Import fixes

* Updating package.json

* Gulpfile script changes

* Gulpfile updates

* Rebuilding package with monorepo updates

* Re-running test

* Updating unittest

* Updating test dependencies

* Updating DESCRIPTION and .Rbuildignore forchecks

* Updating function descriptions and NAMESPACE imports

* Fixed examples and updated docs

* Updating version

* Remove references to dash namespace within package

* Update testthat and remove deprecated context calls

* Removed fixup_metadata.R

* Removing more dash namespace references

* Concatenating component function files

* Updating checks

* Merging components into package R files

* Fixing check

* More package cleanup

* testthat 3.0.0

* Rebuilding package

* Fixed conditional for multiple outputs

* Fix no_update test

* Fixing callback_instrumentation test

* Fixing unit test

* Added DBC to Dash R package (#273)

* Adding dbc to dashR namespace

* updated gitignore

* Adding dbc docs and updating gulpfile

* Updating test with dbc

* Moved misc tests and added dbc snapshot

* Fixing test

* fixing id

* Fixed export and test

* Reverting sorted prop order

* Checks updates

* Re-running test

Co-authored-by: Ryan Patrick Kyle <[email protected]>
Co-authored-by: Ryan Patrick Kyle <[email protected]>
Co-authored-by: Nicolas Kruchten <[email protected]>
Co-authored-by: rpkyle <[email protected]>
Co-authored-by: Steve Sperandeo <[email protected]>
HammadTheOne added a commit that referenced this pull request Oct 14, 2021
* contribute test script

* remove version updating in DESCRIPTION

* fix EOL

Co-authored-by: HammadTheOne <[email protected]>

* Add support for user-defined server routes (#225)

* Provide support for script and stylesheet attributes (#226)

* Authenticate on pulls from Docker Hub (#231)

* Add support for callback graph improvements and timing (#224)

* Update CHANGELOG.md

* 189 - Add Pattern Matching Callbacks for Dash R (#228)

* Testing initial implementation

* More testing

* Callback Context Updates

* Updating callback context logic

* Fixing callback returns

* Adding callback args conditional

* Cleanup and additional changes to callback value conditionals

* Comment cleanup

* Added PMC callback validation, removed unnecessary code

* Update R/dependencies.R

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Update R/dependencies.R

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Update R/dependencies.R

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Update R/dependencies.R

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Added build to gitignore

* Updated dependencies.R

* Update boilerplate docs and add wildcard symbols

* Drying up validation code and applying symbol logic

* Update test to use symbols

* Cleaned up code and added allsmaller test example

* Cleaning up redundant code

* Update FUNDING.yml

* Updated callback_args logic and example

* Adding basic unittests, updated validation

* Fixed response for MATCH callbacks

* Added integration test and updated examples for docs

* Added additional integration test

* Formatting and cleanup

* update docs

* Update to-do app

* Add comments to examples

* Change empy vector to character type.

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Update boilerplate text.

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Update tests/integration/callbacks/test_pattern_matching.py

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Update tests/integration/callbacks/test_pattern_matching.py

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Update tests/integration/callbacks/test_pattern_matching.py

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Update tests/integration/callbacks/test_pattern_matching.py

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Update tests/integration/callbacks/test_pattern_matching.py

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Update tests/testthat/test-wildcards.R

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Update wildcards_test.R

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Update wildcards_test.R

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Update wildcards_test.R

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Update wildcards_test.R

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Update wildcards_test.R

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Update wildcards_test.R

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Update wildcards_test.R

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Update wildcards_test.R

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Removed triple colon syntax

* Use seq_along and remove unnecessary unittest

* Update CHANGELOG.md

* Update CHANGELOG.md

* Add support for arbitrary and sorted keys

* Whitespace deleted

* Added integration tests

* Fixing test output

* Fixing flakiness

* Update test_pattern_matching.py

* Update test_pattern_matching.py

* Updating boilerplate text and test with generalized keys

* Minor test fixes

Co-authored-by: Ryan Patrick Kyle <[email protected]>
Co-authored-by: Nicolas Kruchten <[email protected]>
Co-authored-by: rpkyle <[email protected]>

* Fixing Null error with glue::glue interpolation (#233)

* Fixing NULL error with glue interpolation

* Update utils.R

* Update utils.R

* Update CHANGELOG.md

* Update dash-renderer to v1.8.2 (#234)

* bump dash-renderer to v1.8.2

* Update CHANGELOG.md

* add note about update to dash-renderer

* Fixing flaky test

* bump package version to v0.8.0

* Update R/dash.R

Co-authored-by: HammadTheOne <[email protected]>

* Update tests/testthat/test-wildcards.R

Co-authored-by: HammadTheOne <[email protected]>

* Update DESCRIPTION

Co-authored-by: HammadTheOne <[email protected]>

* add PMC example

* update documentation

* update CHANGELOG release date

* 🔨 PMC docs refactor

* Update tests/integration/callbacks/test_pattern_matching.py

Co-authored-by: HammadTheOne <[email protected]>

* Update tests/integration/callbacks/test_pattern_matching.py

Co-authored-by: HammadTheOne <[email protected]>

* Update tests/integration/callbacks/test_pattern_matching.py

Co-authored-by: HammadTheOne <[email protected]>

* add import of glue

* add glue to imports.R

* fix line length issue

* Fix setCallbackContext for wildcard and ordinary inputs (#237)

* Update setCallbackContext

* Adding graphs test

* Slight fix

* bump version and update CHANGELOG

* Less flaky test

Co-authored-by: rpkyle <[email protected]>

* bump dependency versions

* update CHANGELOG

* update dash-renderer to v1.8.3

* update CHANGELOG

* Favicon fix (#240)

* Adding default favicon

* Removing redundant codeblock

* Added default favicon

* Minor fix to requests prefix

* Update CHANGELOG.md

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Added simple test

* Fixed typo

* Fixed typo

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Remove context reference from CircleCI (#241)

* Dash R Core Package Unification (#243)

* Initialize npm and gulpfile in repo

* Adding directory structure

* Initial implementation of unification script

* Minor grep fixes

* Fixed DESCRIPTION imports

* Added updated dash-table deps

* Regex for version numbers

* Cut import entries from NAMESPACE

* Remove && include(dashr)

* Removing gulp-asset artifacts and rebuilding complete package

* Removing unnecessary files

* fix: remove html, core pkgs from tests

* fix: update script tags unit test

* Revert R6 import

* Add temporary collate

* Update README examples

* Scrubbing imports

* More import scrubbing

* Package development updates

* Update gitignore and namespace

* Updated gulpfile jobs

* Updated all dependencies

* Added templates for namespace/internal exports

* Update internal, namespace, and gulpfile cleanup

* Fix dependency sourcing

* Linting

* Adding job for asset retrieval and deletion

* Minor src change

* Added error handling

* Fixing favicon bug

* chore: use shallow clone

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Added deprecation warning if dcc, html, or table packages are attached (#249)

* Added deprecation warning

* Update R/dash.R

Co-authored-by: Ryan Patrick Kyle <[email protected]>

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Update `highlight.js` dependency for dash-table (#262)

* Updating gulpfile

* Updating dependencies for dash components

* Updated highlight.js dependency source

* Add Dash 2 layout syntax wrappers and html tags (#265)

* Added add_meta helper

* Add helper functions and export pipe

* Adding in tags wrapper

* Updating ci config

* Updated CHANGELOG

* Updated circleci to include rust package manager (cargo).

* Removed dashr command in circleci config.

* Updating changelog

* added basic test

* Cleaning up Dash 2 references and duplication

* Updated documentation

* Allow conditional UI

* Add meta tag check

* More cleanup

* Last bit of cleanup

Co-authored-by: Steve Sperandeo <[email protected]>

* Fix suppress_callback_exceptions config (#268)

* Add config key

* Fixing CI

* Simplified callback syntax and addtional utility functions (#270)

* Tag updates

* Added RStudio dash snippet

* Added simple_table

* Added flexible callbacks

* Documentation and NAMESPACE updates

* Updated DESCRIPTION

* Adding unittests

* Adding context tags to tests

* Updated CHANGELOG

* Update monorepo and rebuild package (#271)

* Updating gulpfile and package.json

* More package.json and linting updates

* Adding in simplified callback updates/tests

* Import fixes

* Updating package.json

* Gulpfile script changes

* Gulpfile updates

* Rebuilding package with monorepo updates

* Re-running test

* Updating unittest

* Updating test dependencies

* Updating DESCRIPTION and .Rbuildignore forchecks

* Updating function descriptions and NAMESPACE imports

* Fixed examples and updated docs

* Updating version

* Remove references to dash namespace within package

* Update testthat and remove deprecated context calls

* Removed fixup_metadata.R

* Removing more dash namespace references

* Concatenating component function files

* Updating checks

* Merging components into package R files

* Fixing check

* More package cleanup

* testthat 3.0.0

* Rebuilding package

* Fixed conditional for multiple outputs

* Fix no_update test

* Fixing callback_instrumentation test

* Fixing unit test

* Added DBC to Dash R package (#273)

* Adding dbc to dashR namespace

* updated gitignore

* Adding dbc docs and updating gulpfile

* Updating test with dbc

* Moved misc tests and added dbc snapshot

* Fixing test

* fixing id

* Fixed export and test

* Reverting sorted prop order

* Checks updates

* Re-running test

* Remove html exports and update tags (#274)

* Updating tags and html exports

* Re-running tests

* Updating tag generation

* Re-running tests

* Updating tests to use new html list syntax

* Adjusting tests

* Re-run tests again

* Updating tests

* Wrapping up test fixes

* Percy test

Co-authored-by: Ryan Patrick Kyle <[email protected]>
Co-authored-by: Ryan Patrick Kyle <[email protected]>
Co-authored-by: Nicolas Kruchten <[email protected]>
Co-authored-by: rpkyle <[email protected]>
Co-authored-by: Steve Sperandeo <[email protected]>
HammadTheOne added a commit that referenced this pull request Nov 1, 2021
* contribute test script

* remove version updating in DESCRIPTION

* fix EOL

Co-authored-by: HammadTheOne <[email protected]>

* Add support for user-defined server routes (#225)

* Provide support for script and stylesheet attributes (#226)

* Authenticate on pulls from Docker Hub (#231)

* Add support for callback graph improvements and timing (#224)

* Update CHANGELOG.md

* 189 - Add Pattern Matching Callbacks for Dash R (#228)

* Testing initial implementation

* More testing

* Callback Context Updates

* Updating callback context logic

* Fixing callback returns

* Adding callback args conditional

* Cleanup and additional changes to callback value conditionals

* Comment cleanup

* Added PMC callback validation, removed unnecessary code

* Update R/dependencies.R

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Update R/dependencies.R

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Update R/dependencies.R

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Update R/dependencies.R

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Added build to gitignore

* Updated dependencies.R

* Update boilerplate docs and add wildcard symbols

* Drying up validation code and applying symbol logic

* Update test to use symbols

* Cleaned up code and added allsmaller test example

* Cleaning up redundant code

* Update FUNDING.yml

* Updated callback_args logic and example

* Adding basic unittests, updated validation

* Fixed response for MATCH callbacks

* Added integration test and updated examples for docs

* Added additional integration test

* Formatting and cleanup

* update docs

* Update to-do app

* Add comments to examples

* Change empy vector to character type.

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Update boilerplate text.

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Update tests/integration/callbacks/test_pattern_matching.py

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Update tests/integration/callbacks/test_pattern_matching.py

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Update tests/integration/callbacks/test_pattern_matching.py

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Update tests/integration/callbacks/test_pattern_matching.py

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Update tests/integration/callbacks/test_pattern_matching.py

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Update tests/testthat/test-wildcards.R

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Update wildcards_test.R

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Update wildcards_test.R

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Update wildcards_test.R

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Update wildcards_test.R

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Update wildcards_test.R

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Update wildcards_test.R

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Update wildcards_test.R

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Update wildcards_test.R

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Removed triple colon syntax

* Use seq_along and remove unnecessary unittest

* Update CHANGELOG.md

* Update CHANGELOG.md

* Add support for arbitrary and sorted keys

* Whitespace deleted

* Added integration tests

* Fixing test output

* Fixing flakiness

* Update test_pattern_matching.py

* Update test_pattern_matching.py

* Updating boilerplate text and test with generalized keys

* Minor test fixes

Co-authored-by: Ryan Patrick Kyle <[email protected]>
Co-authored-by: Nicolas Kruchten <[email protected]>
Co-authored-by: rpkyle <[email protected]>

* Fixing Null error with glue::glue interpolation (#233)

* Fixing NULL error with glue interpolation

* Update utils.R

* Update utils.R

* Update CHANGELOG.md

* Update dash-renderer to v1.8.2 (#234)

* bump dash-renderer to v1.8.2

* Update CHANGELOG.md

* add note about update to dash-renderer

* Fixing flaky test

* bump package version to v0.8.0

* Update R/dash.R

Co-authored-by: HammadTheOne <[email protected]>

* Update tests/testthat/test-wildcards.R

Co-authored-by: HammadTheOne <[email protected]>

* Update DESCRIPTION

Co-authored-by: HammadTheOne <[email protected]>

* add PMC example

* update documentation

* update CHANGELOG release date

* 🔨 PMC docs refactor

* Update tests/integration/callbacks/test_pattern_matching.py

Co-authored-by: HammadTheOne <[email protected]>

* Update tests/integration/callbacks/test_pattern_matching.py

Co-authored-by: HammadTheOne <[email protected]>

* Update tests/integration/callbacks/test_pattern_matching.py

Co-authored-by: HammadTheOne <[email protected]>

* add import of glue

* add glue to imports.R

* fix line length issue

* Fix setCallbackContext for wildcard and ordinary inputs (#237)

* Update setCallbackContext

* Adding graphs test

* Slight fix

* bump version and update CHANGELOG

* Less flaky test

Co-authored-by: rpkyle <[email protected]>

* bump dependency versions

* update CHANGELOG

* update dash-renderer to v1.8.3

* update CHANGELOG

* Favicon fix (#240)

* Adding default favicon

* Removing redundant codeblock

* Added default favicon

* Minor fix to requests prefix

* Update CHANGELOG.md

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Added simple test

* Fixed typo

* Fixed typo

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Remove context reference from CircleCI (#241)

* Dash R Core Package Unification (#243)

* Initialize npm and gulpfile in repo

* Adding directory structure

* Initial implementation of unification script

* Minor grep fixes

* Fixed DESCRIPTION imports

* Added updated dash-table deps

* Regex for version numbers

* Cut import entries from NAMESPACE

* Remove && include(dashr)

* Removing gulp-asset artifacts and rebuilding complete package

* Removing unnecessary files

* fix: remove html, core pkgs from tests

* fix: update script tags unit test

* Revert R6 import

* Add temporary collate

* Update README examples

* Scrubbing imports

* More import scrubbing

* Package development updates

* Update gitignore and namespace

* Updated gulpfile jobs

* Updated all dependencies

* Added templates for namespace/internal exports

* Update internal, namespace, and gulpfile cleanup

* Fix dependency sourcing

* Linting

* Adding job for asset retrieval and deletion

* Minor src change

* Added error handling

* Fixing favicon bug

* chore: use shallow clone

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Added deprecation warning if dcc, html, or table packages are attached (#249)

* Added deprecation warning

* Update R/dash.R

Co-authored-by: Ryan Patrick Kyle <[email protected]>

Co-authored-by: Ryan Patrick Kyle <[email protected]>

* Update `highlight.js` dependency for dash-table (#262)

* Updating gulpfile

* Updating dependencies for dash components

* Updated highlight.js dependency source

* Add Dash 2 layout syntax wrappers and html tags (#265)

* Added add_meta helper

* Add helper functions and export pipe

* Adding in tags wrapper

* Updating ci config

* Updated CHANGELOG

* Updated circleci to include rust package manager (cargo).

* Removed dashr command in circleci config.

* Updating changelog

* added basic test

* Cleaning up Dash 2 references and duplication

* Updated documentation

* Allow conditional UI

* Add meta tag check

* More cleanup

* Last bit of cleanup

Co-authored-by: Steve Sperandeo <[email protected]>

* Fix suppress_callback_exceptions config (#268)

* Add config key

* Fixing CI

* Simplified callback syntax and addtional utility functions (#270)

* Tag updates

* Added RStudio dash snippet

* Added simple_table

* Added flexible callbacks

* Documentation and NAMESPACE updates

* Updated DESCRIPTION

* Adding unittests

* Adding context tags to tests

* Updated CHANGELOG

* Update monorepo and rebuild package (#271)

* Updating gulpfile and package.json

* More package.json and linting updates

* Adding in simplified callback updates/tests

* Import fixes

* Updating package.json

* Gulpfile script changes

* Gulpfile updates

* Rebuilding package with monorepo updates

* Re-running test

* Updating unittest

* Updating test dependencies

* Updating DESCRIPTION and .Rbuildignore forchecks

* Updating function descriptions and NAMESPACE imports

* Fixed examples and updated docs

* Updating version

* Remove references to dash namespace within package

* Update testthat and remove deprecated context calls

* Removed fixup_metadata.R

* Removing more dash namespace references

* Concatenating component function files

* Updating checks

* Merging components into package R files

* Fixing check

* More package cleanup

* testthat 3.0.0

* Rebuilding package

* Fixed conditional for multiple outputs

* Fix no_update test

* Fixing callback_instrumentation test

* Fixing unit test

* Added DBC to Dash R package (#273)

* Adding dbc to dashR namespace

* updated gitignore

* Adding dbc docs and updating gulpfile

* Updating test with dbc

* Moved misc tests and added dbc snapshot

* Fixing test

* fixing id

* Fixed export and test

* Reverting sorted prop order

* Checks updates

* Re-running test

* Remove html exports and update tags (#274)

* Updating tags and html exports

* Re-running tests

* Updating tag generation

* Re-running tests

* Updating tests to use new html list syntax

* Adjusting tests

* Re-run tests again

* Updating tests

* Wrapping up test fixes

* Percy test

* CRAN submission updates

* Fix callback outputs with short ID's (#280)

* CRAN submission updates

* Callback output ID validation fixes

* Updating duplicate callbacks test

* Security updates

* Update R/utils.R

Co-authored-by: Dean Attali <[email protected]>

* Updating error message

Co-authored-by: Dean Attali <[email protected]>

* Updating README link

Co-authored-by: Ryan Patrick Kyle <[email protected]>
Co-authored-by: Ryan Patrick Kyle <[email protected]>
Co-authored-by: Nicolas Kruchten <[email protected]>
Co-authored-by: rpkyle <[email protected]>
Co-authored-by: Steve Sperandeo <[email protected]>
Co-authored-by: Dean Attali <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants