Browse Source

fetch GitHubSecuritySettingsData

kaori 3 years ago
parent
commit
62e0086fe4
1 changed files with 16 additions and 19 deletions
  1. 16 19
      packages/app/src/components/Admin/Security/GitHubSecuritySetting.jsx

+ 16 - 19
packages/app/src/components/Admin/Security/GitHubSecuritySetting.jsx

@@ -1,5 +1,4 @@
-/* eslint-disable react/no-danger */
-import React from 'react';
+import React, { useEffect } from 'react';
 
 import PropTypes from 'prop-types';
 
@@ -15,23 +14,21 @@ import GitHubSecuritySettingContents from './GitHubSecuritySettingContents';
 let retrieveErrors = null;
 function GitHubSecurityManagement(props) {
   const { adminGitHubSecurityContainer } = props;
-  // if (adminGitHubSecurityContainer.state.githubClientId === adminGitHubSecurityContainer.dummyGithubClientId) {
-  //   throw (async() => {
-  //     try {
-  //       await adminGitHubSecurityContainer.retrieveSecurityData();
-  //     }
-  //     catch (err) {
-  //       const errs = toArrayIfNot(err);
-  //       toastError(errs);
-  //       retrieveErrors = errs;
-  //       adminGitHubSecurityContainer.setState({ githubClientId: adminGitHubSecurityContainer.dummyGithubClientIdForError });
-  //     }
-  //   })();
-  // }
-
-  // if (adminGitHubSecurityContainer.state.githubClientId === adminGitHubSecurityContainer.dummyGithubClientIdForError) {
-  //   throw new Error(`${retrieveErrors.length} errors occured`);
-  // }
+
+  useEffect(() => {
+    const fetchGitHubSecuritySettingsData = async() => {
+      await adminGitHubSecurityContainer.retrieveSecurityData();
+    };
+
+    try {
+      fetchGitHubSecuritySettingsData();
+    }
+    catch (err) {
+      const errs = toArrayIfNot(err);
+      toastError(errs);
+      logger.error(errs);
+    }
+  }, [adminGitHubSecurityContainer]);
 
   return <GitHubSecuritySettingContents />;
 }