-
Notifications
You must be signed in to change notification settings - Fork 9
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
I404 backend update #415
I404 backend update #415
Conversation
ccm_web/Dockerfile.openshift
Outdated
@@ -1,5 +1,5 @@ | |||
# Build stage (build client and compile server to JS) |
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 will be delete once we have the issue #412 merged
@@ -42,7 +42,7 @@ const getDelete = (body: string): RequestInit => { | |||
|
|||
// This currently assumes all put requests have a JSON payload and receive a JSON response. | |||
const getPut = (body: string): RequestInit => { | |||
const headers: string[][] = [['Content-Type', jsonMimeType], ['Accept', jsonMimeType]] |
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.
I found these issues with typescript update
@@ -19,95 +19,95 @@ | |||
"author": "", | |||
"license": "ISC", | |||
"dependencies": { | |||
"@kth/canvas-api": "^4.2.2", |
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.
got
and p-limit
is not updated since they are ESM module and don't backport support for CJS
96c606b
to
88d09ff
Compare
88d09ff
to
b1b9718
Compare
@jaydonkrooss , I have pointed this branch to tl-its-umich-edu:2024-03-01-dep-update as we discussed. Once this merged to that branch we can have your #411 branch get the latest from this. As this has the latest typescript |
efe3435
into
tl-its-umich-edu:2024-03-01-dep-update
Fixes #404
Do a NPM install and delete the node_modules for new package.json update
ncu
package for checking what package are updated.got
andp-limit
is not updated since they are ESM module and don't backport support for CJSTest Plan: Look for plan over here
Dep_update_test_plan: https://docs.google.com/spreadsheets/d/1kFDRcji_RySw_DtF-ZunvKhSZ9rYNMaPoVGJxVbje4c/edit#gid=0
Testing support: https://drive.google.com/drive/folders/1Ny6KgyoG-N9sOwo12gEXTOEPkB22VlUB
CCM doc: https://drive.google.com/drive/folders/1jFM54JYJVnOhUTQbZNeGQl5yW7F_uVwl