diff --git a/src/components/Flashpack/Add/index.jsx b/src/components/Flashpack/Add/index.jsx index 69d527a..a0b1df1 100644 --- a/src/components/Flashpack/Add/index.jsx +++ b/src/components/Flashpack/Add/index.jsx @@ -1,8 +1,11 @@ import { Button, + IconButton, TextField } from "@material-ui/core"; import React, { useEffect, useState } from "react"; +import AddCircleIcon from "@material-ui/icons/AddCircle"; +import DeleteIcon from "@material-ui/icons/Delete"; import { Redirect } from "react-router"; import { logger } from "../../../services/monitoring"; import { useStatusContext } from "../../Contexts/StatusContext"; @@ -31,9 +34,16 @@ const MainForm = () => { useEffect(() => { setTitle(`Add Flashpack Version`); setSitePath([ + { + label: "Tools", + link: "/tools/flashpacks", + }, + { + label: "Flashpack Versions", + link: "/tools/flashpacks", + }, { label: `Add Flashpack Version`, - link: `/tools/flashpack/add`, }, ]); // eslint-disable-next-line react-hooks/exhaustive-deps @@ -57,20 +67,21 @@ const MainForm = () => { const carFlashpackVersions = [] for (let i = 0; i < mappingInputs.length; i++) { - carFlashpackVersions.push({ - "car_model": carModel, - "car_year": parseInt(carYear), - "flashpack": flashpack, - "car_ecu_name": mappingInputs[i].ecuName, - "car_ecu_version": mappingInputs[i].ecuVersion, - }) + mappingInputs[i] && mappingInputs[i].ecuName && mappingInputs[i].ecuVersion && + carFlashpackVersions.push({ + "car_model": carModel, + "car_year": parseInt(carYear), + "flashpack": flashpack, + "car_ecu_name": mappingInputs[i].ecuName, + "car_ecu_version": mappingInputs[i].ecuVersion, + }) } const result = await addFlashpackVersion(carModel, parseInt(carYear), flashpack, carFlashpackVersions, token); if (!result || result.error) return; setMessage(`Added ${carYear} ${carModel} ${flashpack}`); - setRedirect(`/tools/flashpacks`); + setRedirect(`/tools/flashpack/${carModel}/${carYear}/${flashpack}`); } catch (e) { setMessage(e.message); logger.warn(e.stack); @@ -180,14 +191,27 @@ const MainForm = () => { type="text" /> {mappingInputs.length > 1 && ( - + handleDeleteMappingInput(index)} + aria-label={`Delete`} + size="small" + color="primary" + > + + )} {index === mappingInputs.length - 1 && ( - + handleAddMappingInput()} + aria-label={`Add`} + size="small" + color="primary" + > + + )} ))} -
{JSON.stringify(mappingInputs)}