Improved error handling

openml · Feb 1, 2024 · f5aa599 · f5aa599
1 parent 282ac7f
commit f5aa599
Show file tree

Hide file tree

Showing 4 changed files with 56 additions and 30 deletions.
diff --git a/app/src/pages/d/[dataId].js b/app/src/pages/d/[dataId].js
@@ -77,28 +77,25 @@ const VersionChip = styled(Chip)`
 // Loads the information about the dataset from ElasticSearch
 // Also loads the translations for the page
 export async function getStaticProps({ params, locale }) {
- try {
- // Fetch necessary data for the dataset page using params.dataId
- const data = await getItem("data", params.dataId);
+ let data = null;
+ let error = null;
 
- return {
- props: {
- data,
- error: null, // No error occurred
- ...(await serverSideTranslations(locale)),
- },
- };
+ try {
+ data = await getItem("data", params.dataId);
  } catch (error) {
  console.error("Error in getStaticProps:", error);
-
- return {
- props: {
- data: null, // No data due to error
- error: "Server is not responding.",
- ...(await serverSideTranslations(locale)),
- },
- };
+ error = "Server is not responding.";
  }
+
+ const translations = await serverSideTranslations(locale);
+
+ return {
+ props: {
+ data,
+ error,
+ ...translations,
+ },
+ };
 }
 
 const ActionButtons = ({ buttons }) => {

diff --git a/app/src/pages/f/[flowId].js b/app/src/pages/f/[flowId].js
@@ -35,14 +35,23 @@ export async function getStaticPaths() {
 }
 
 export async function getStaticProps({ params, locale }) {
- // Fetch necessary data for the flow page using params.flowId
- const data = await getItem("flow", params.flowId);
+ let data = null;
+ let error = null;
+
+ try {
+ data = await getItem("flow", params.flowId);
+ } catch (error) {
+ console.error("Error in getStaticProps:", error);
+ error = "Server is not responding.";
+ }
+
+ const translations = await serverSideTranslations(locale);
 
  return {
  props: {
  data,
- // pass the translation props to the page component
- ...(await serverSideTranslations(locale)),
+ error,
+ ...translations,
  },
  };
 }

diff --git a/app/src/pages/r/[runId].js b/app/src/pages/r/[runId].js
@@ -36,13 +36,23 @@ export async function getStaticPaths() {
 }
 
 export async function getStaticProps({ params, locale }) {
- // Fetch necessary data for the dataset page using params.dataId
- const data = await getItem("run", params.runId);
+ let data = null;
+ let error = null;
+
+ try {
+ data = await getItem("run", params.runId);
+ } catch (error) {
+ console.error("Error in getStaticProps:", error);
+ error = "Server is not responding.";
+ }
+
+ const translations = await serverSideTranslations(locale);
+
  return {
  props: {
  data,
- // pass the translation props to the page component
- ...(await serverSideTranslations(locale)),
+ error,
+ ...translations,
  },
  };
 }

diff --git a/app/src/pages/t/[taskId].js b/app/src/pages/t/[taskId].js
@@ -35,13 +35,23 @@ export async function getStaticPaths() {
 }
 
 export async function getStaticProps({ params, locale }) {
- // Fetch necessary data for the task page using params.dataId
- const data = await getItem("task", params.taskId);
+ let data = null;
+ let error = null;
+
+ try {
+ data = await getItem("task", params.taskId);
+ } catch (error) {
+ console.error("Error in getStaticProps:", error);
+ error = "Server is not responding.";
+ }
+
+ const translations = await serverSideTranslations(locale);
+
  return {
  props: {
  data,
- // pass the translation props to the page component
- ...(await serverSideTranslations(locale)),
+ error,
+ ...translations,
  },
  };
 }