-
-
Notifications
You must be signed in to change notification settings - Fork 289
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
Bump dependencies versions #921
Bump dependencies versions #921
Conversation
65f6af4
to
dbc4c34
Compare
e9dd443
to
168f739
Compare
168f739
to
0e2c601
Compare
if (ev.lengthComputable) { | ||
progress.setUploadProgress(ev.loaded / ev.total * 100); // percentage complete | ||
function onUploadProgress(ev: AxiosProgressEvent) { | ||
if (ev.event.lengthComputable) { |
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.
FWIW, I achieved to hit a breakpoint here. The workspaceId
equals to "unsaved"
when the user is not logged in but is allowed to upload documents (temporary).
I could inspect the properties of ev.event
and they have consistent values.
catw app/client/*.css app/client/*/*.css -o static/bundle.css -v & webpack --config $WEBPACK_CONFIG --mode development --watch & | ||
css_files="app/client/**/*.css" | ||
chokidar "${css_files}" -c "bash -O globstar -c 'cat ${css_files} > static/bundle.css'" & | ||
webpack --config $WEBPACK_CONFIG --mode development --watch & |
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.
This is a pretty cool way to replace the abandoned catw
package.
@jordigh Thanks for your review. Do you wait for another review before merging? Otherwise I do not have the rights to merge myself. |
@fflorent I need to first verify deploying to our own Grist SAAS before I merge. |
@jordigh Sounds good! 👍 |
Thanks for your patience. After some internal testing, we are satisfied with the state of this PR. |
No description provided.