-
Notifications
You must be signed in to change notification settings - Fork 2
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
Upload merged #77
base: main
Are you sure you want to change the base?
Upload merged #77
Conversation
tasks.py
Outdated
@@ -372,6 +387,7 @@ def show_outdated(ctx: Context): | |||
def upgrade(ctx: Context): | |||
"""Upgrade Python and JS packages""" | |||
ctx.run("poetry update", pty=True) | |||
copy_statics(ctx) |
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.
And npm?! Or what are you copying?
@@ -26,6 +26,7 @@ | |||
"debug_toolbar", | |||
"debug_permissions", | |||
"django_browser_reload", | |||
"adit.upload.apps.UploadConfig", |
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.
Only in development!?
adit/core/widgets.py
Outdated
@@ -13,7 +13,9 @@ def create_option(self, name, value, label, selected, index, subindex=None, attr | |||
dicom_node = value.instance | |||
if dicom_node.node_type == DicomNode.NodeType.SERVER: | |||
option["attrs"]["data-node_type"] = "server" | |||
option["attrs"]["node_id"] = dicom_node.id |
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.
It's not a valid HTML attribute! Add data-
expect(poll(page.locator("button#uploadButton"))).to_be_visible() | ||
expect(poll(page.locator("button#clearButton"))).to_be_visible() | ||
|
||
# page.screenshot(path="./screenshots/before_upload.png") |
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.
Get rid of all unnecessary comments
adit/upload/static/upload/upload.js
Outdated
document.getElementById("uploadCompleteText").style.display = "none"; | ||
}, | ||
clearFiles: function () { | ||
var inputEl = document.getElementById("fileselector"); |
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.
document?!
adit/upload/static/upload/upload.js
Outdated
document.getElementById("uploadCompleteText").style.display = | ||
"inline-block"; | ||
}, 3000); | ||
// Wait for 3 seconds (3000 milliseconds) |
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.
What is the purpose of this comment? It's pretty clear from the code
adit/upload/static/upload/upload.js
Outdated
"inline-block"; | ||
} | ||
|
||
console.log("Upload process finished"); |
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.
Don't use log if you want to leave it in the code. Use info, for example.
adit/upload/static/upload/upload.js
Outdated
patientBirthdates.size <= 1 | ||
); | ||
} catch (e) { | ||
//console.log(e); |
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.
?!
adit/upload/static/upload/upload.js
Outdated
} | ||
}) | ||
.catch(function (error) { | ||
//console.log(`Error: ${error.reason_phrase}`); |
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.
Please cleanup your code :-P
Otherwise all those manage commands would also run multiple times
No description provided.