diff --git a/portal-ui/src/screens/Console/Buckets/BucketDetails/AddLifecycleModal.tsx b/portal-ui/src/screens/Console/Buckets/BucketDetails/AddLifecycleModal.tsx index 134243a26..eb8315de0 100644 --- a/portal-ui/src/screens/Console/Buckets/BucketDetails/AddLifecycleModal.tsx +++ b/portal-ui/src/screens/Console/Buckets/BucketDetails/AddLifecycleModal.tsx @@ -32,7 +32,6 @@ import InputBoxWrapper from "../../Common/FormComponents/InputBoxWrapper/InputBo import FormSwitchWrapper from "../../Common/FormComponents/FormSwitchWrapper/FormSwitchWrapper"; import ModalWrapper from "../../Common/ModalWrapper/ModalWrapper"; import api from "../../../../common/api"; -import DateSelector from "../../Common/FormComponents/DateSelector/DateSelector"; import SelectWrapper from "../../Common/FormComponents/SelectWrapper/SelectWrapper"; import QueryMultiSelector from "../../Common/FormComponents/QueryMultiSelector/QueryMultiSelector"; import RadioGroupSelector from "../../Common/FormComponents/RadioGroupSelector/RadioGroupSelector"; @@ -98,12 +97,8 @@ const AddLifecycleModal = ({ const [NCExpirationDays, setNCExpirationDays] = useState("0"); const [NCTransitionDays, setNCTransitionDays] = useState("0"); const [ilmType, setIlmType] = useState("expiry"); - const [expiryType, setExpiryType] = useState("date"); const [expiryDays, setExpiryDays] = useState("0"); - const [expiryDate, setExpiryDate] = useState(""); const [transitionDays, setTransitionDays] = useState("0"); - const [transitionDate, setTransitionDate] = useState(""); - const [transitionType, setTransitionType] = useState("date"); const [isFormValid, setIsFormValid] = useState(false); useEffect(() => { @@ -137,69 +132,30 @@ const AddLifecycleModal = ({ useEffect(() => { let valid = true; - if (ilmType === "expiry") { - if (expiryType === "date" && expiryDate === "") { - valid = false; - } - if (expiryType === "days" && parseInt(expiryDays) < 1) { - valid = false; - } - } else { - if (transitionType === "date" && transitionDate === "") { - valid = false; - } - if (transitionType === "days" && parseInt(transitionDays) < 1) { - valid = false; - } - + if (ilmType !== "expiry") { if (storageClass === "") { valid = false; } } setIsFormValid(valid); - }, [ - ilmType, - expiryType, - expiryDate, - expiryDays, - transitionType, - transitionDate, - transitionDays, - storageClass, - ]); + }, [ilmType, expiryDays, transitionDays, storageClass]); const addRecord = () => { let rules = {}; if (ilmType === "expiry") { - let expiry = {}; - - if (expiryType === "date") { - expiry = { - expiry_date: `${expiryDate}T23:59:59Z`, - }; - } else { - expiry = { - expiry_days: parseInt(expiryDays), - }; - } + let expiry = { + expiry_days: parseInt(expiryDays), + }; rules = { ...expiry, noncurrentversion_expiration_days: parseInt(NCExpirationDays), }; } else { - let transition = {}; - - if (transitionType === "date") { - transition = { - transition_date: `${transitionDate}T23:59:59Z`, - }; - } else { - transition = { - transition_days: parseInt(transitionDays), - }; - } + let transition = { + transition_days: parseInt(transitionDays), + }; rules = { ...transition, @@ -285,62 +241,20 @@ const AddLifecycleModal = ({ {ilmType === "expiry" ? ( - + e: React.ChangeEvent ) => { - setExpiryType(e.target.value as string); + setExpiryDays(e.target.value); }} - selectorOptions={[ - { value: "date", label: "Date" }, - { value: "days", label: "Days" }, - ]} + label="Expiry Days" + value={expiryDays} + min="0" /> - - {expiryType === "date" ? ( - - { - if (isValid) { - setExpiryDate(date); - } - }} - /> - - ) : ( - - - ) => { - setExpiryDays(e.target.value); - }} - label="Expiry Days" - value={expiryDays} - min="0" - /> - - )} - - + e: React.ChangeEvent ) => { - setTransitionType(e.target.value as string); + setTransitionDays(e.target.value); }} - selectorOptions={[ - { value: "date", label: "Date" }, - { value: "days", label: "Days" }, - ]} + label="Transition Days" + value={transitionDays} + min="0" /> - - {transitionType === "date" ? ( - - { - if (isValid) { - setTransitionDate(date); - } - }} - /> - - ) : ( - - - ) => { - setTransitionDays(e.target.value); - }} - label="Transition Days" - value={transitionDays} - min="0" - /> - - )} -