Merge pull request #562 from battlecode/avatar-and-resume-size-check

frontend avatar and resume file size check
battlecode · Jan 25, 2023 · ab69f10 · ab69f10
2 parents 3ac9ee7 + e1cadce
commit ab69f10
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 3 deletions.
diff --git a/frontend/src/components/avatarUpload.js b/frontend/src/components/avatarUpload.js
@@ -28,10 +28,10 @@ class AvatarUpload extends Component {
       } else {
         this.setState({ status: "failure" });
       }
-      setTimeout(() => {
-        this.setState({ status: "waiting" });
-      }, 2000);
     });
+    setTimeout(() => {
+      this.setState({ status: "waiting" });
+    }, 2000);
   }
 
   render() {

diff --git a/frontend/src/views/account.js b/frontend/src/views/account.js
@@ -102,6 +102,16 @@ class Account extends Component {
   }
 
   uploadResume = () => {
+    const fileSize = this.state.selectedResumeFile.size / 1024 / 1024; // in MiB
+    if (fileSize > 5) {
+      let alert_message =
+        "Ensure this file size is not greater than 5 MiB. Your current file size is " +
+        Number(fileSize.toPrecision(3)) +
+        " MiB.";
+      this.setState({ alert_message: alert_message });
+      this.setState({ resume_status: "waiting" });
+      return;
+    }
     this.setState({ resume_status: "loading" });
     Api.resumeUpload(this.state.selectedResumeFile, (success) => {
       if (success) {
@@ -117,6 +127,15 @@ class Account extends Component {
   };
 
   uploadAvatar = (selected_file, callback) => {
+    const fileSize = selected_file.size / 1024 / 1024; // in MiB
+    if (fileSize > 5) {
+      let alert_message =
+        "Ensure this file size is not greater than 5 MiB. Your current file size is " +
+        Number(fileSize.toPrecision(3)) +
+        " MiB.";
+      this.setState({ alert_message: alert_message });
+      return;
+    }
     Api.avatarUpload(selected_file, (response, success) => {
       if (success) {
         this.props.updateBaseState();