-
-
Notifications
You must be signed in to change notification settings - Fork 12
Update dependencies #303
base: dev
Are you sure you want to change the base?
Update dependencies #303
Conversation
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.
Wooh more dependency updates! Slightly less wooh is that the cookies and editor are borked, meaning I couldn't fully test the pako dependency update either. :(
src/client/user-state.js
Outdated
} | ||
}, false); | ||
|
||
let userState = { | ||
AUTH_CHANGED: 0, | ||
|
||
data: Cookies.getJSON("user_data") | ||
data: JSON.parse(Cookies.get("user_data")) |
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.
"reconnecting-websocket": "^4.4.0", | ||
"request": "^2.88.0", | ||
"request-promise-native": "^1.0.7", | ||
"semver": "^7.1.0", | ||
"stats-js": "^1.0.0", | ||
"styled-components": "^5.1.0", | ||
"three": "^0.123.0", | ||
"three": "^0.132.0", |
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.
Heck I'm so smart, probably didn't even rebuild the client after bumping version numbers but before testing. |
(Leaving it unset will be unsupported in the future, causing the cookie to be rejected)
- Also changed editorLog to have messages appear in the regular browser console too.
@Qaomen Actually before you approve this, try this map: When you load it, the start & starting gate are slightly offset, until you click the world tab, which causes it to snap back to how they should be positioned. |
More dependency updates! Woo! Hopefully this also fixes any open security issues when it comes to dependencies.
Only
Cookies
had breaking changes that affected us, so that logic has been updated.Discord.js
is not updated in this PR because that's done in #301 of course.