This repository was archived by the owner on Jun 4, 2024. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 33
Clientside Callbacks #143
Merged
Merged
Clientside Callbacks #143
Changes from 48 commits
Commits
Show all changes
50 commits
Select commit
Hold shift + click to select a range
e2455d2
:truck: Move `updateRequestQueue` to the top of the function
chriddyp 8edbc0d
:racehorse: clientside hook
chriddyp 5b9934d
:pencil: examples
chriddyp cf888fd
:one: first clientside test
chriddyp b2037be
:package: temporarily install the dash clientside-2 branch to run tests
chriddyp cefc5a2
:wrench: add the `clientside tests to the `test:py` run
chriddyp a33d765
:package: create builds
chriddyp 78f9bdc
:zap: rm sample assets as it fails the linting tests
chriddyp 3394dfd
:rose: prettier
chriddyp 707fca9
:package: new build
chriddyp 42a2484
:see_no_evil: missing import
chriddyp 7380b55
:link: test chained assertions
chriddyp 77b965f
:hand: tell eslint to ignore clientside test js
chriddyp 57d3f08
:boom: handle & test clientside functions that raise an error
chriddyp fb51d1f
:wrench: eslint ignore
chriddyp f8eb31a
:rose: prettier
chriddyp 7482e41
:question: try combining tests into single file
chriddyp b942a18
:truck: move clientside update logic into a function
chriddyp d868fb1
:1234: handle multiple outputs clientside
chriddyp 0032132
:white_check_mark: test multiple outputs
chriddyp a2cb0c7
:rose: prettier
chriddyp 9548653
:see_no_evil: typo
chriddyp 9cf8b05
:pencil: rename `client` to `clientside` everywhere
chriddyp 2ff50f1
:factory: simple to change to trigger a new circle build
chriddyp f89975c
:pencil: :clock1: rename and handle & test promises
chriddyp 0974e66
:golf: inputs are always defined
chriddyp e54e365
:see_no_evil: typos all day
chriddyp 87b686e
:name_badge: everything in a namespace
chriddyp 872a2b7
:rose: prettier
chriddyp 0c6ebfc
:palm_tree:
chriddyp efdff7b
:airplane: forgot to export
chriddyp 20d0168
:hamburger: sloppy copy pasting
chriddyp 1a5bf70
:see_no_evil:
chriddyp 6ebc26e
:pencil: CHANGELOG
chriddyp 7811b5f
:book: `CLIENTSIDE_ERROR` constant
chriddyp fb4b1d7
:nut_and_bolt: `if ({})` is `true`
chriddyp c2a7c37
:clock1: treat promises like the other functions - don't break the app
chriddyp 714d30f
:see_no_evil: another typo
chriddyp 0065a80
:1234: no longer use `ramda` namespace in test
chriddyp 28590e7
:hear_no_evil: i make this typo so frequently
chriddyp 374493d
:hocho: unify the test methods
chriddyp e06cd88
:pencil: reformat clientside tests to use existing functions
chriddyp 67b3a6a
:rose: i will never get prettier right on the first try
chriddyp 86111aa
:shower: eslint
chriddyp 7f8b9e7
:hocho: remove sample examples
chriddyp 26a5d28
:zap: rm unncessary comment
chriddyp cfa00da
:package: new build
chriddyp ed2cc64
:package: new build
chriddyp dd862bb
:zap: simpler API check as according to https://github.com/plotly/das…
chriddyp 149820e
:lock: clientside on master now
chriddyp File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -43,6 +43,7 @@ jobs: | |
git clone [email protected]:plotly/dash-html-components.git | ||
git clone [email protected]:plotly/dash-table.git | ||
. venv/bin/activate | ||
cd dash && git checkout clientside-2 && cd .. | ||
pip install -e ./dash --quiet | ||
cd dash-core-components && npm install --ignore-scripts && npm run build && pip install -e . && cd .. | ||
cd dash-html-components && npm install --ignore-scripts && npm run build && pip install -e . && cd .. | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -14,3 +14,4 @@ dist | |
|
||
dash_renderer/**/*.min.js | ||
dash_renderer/**/*.dev.js | ||
tests/test_clientside/ |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note: You can also do add
-b clientside-2
in the git clone command above.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
non-⛔️ as this line will disappear when the companion PR merges... but we may benefit 🐎 from using a shallow clone
git clone -depth=1
on CI, in which case-b <branch>
may be the required method.