Always edit script, remove details modal from script page (#26325)

#26323
This commit is contained in:
Dante Catalfamo
2025-02-14 09:31:21 -05:00
committed by GitHub
parent 470766c9ac
commit 7c2ac65dab
2 changed files with 6 additions and 33 deletions
@@ -37,9 +37,7 @@ interface IScriptsProps {
const Scripts = ({ router, currentPage, teamIdForApi }: IScriptsProps) => {
const { isPremiumTier } = useContext(AppContext);
const [showDeleteScriptModal, setShowDeleteScriptModal] = useState(false);
const [showScriptDetailsModal, setShowScriptDetailsModal] = useState(false);
const [showEditScripsModal, setShowEditScriptModal] = useState(false);
const [goBackToScriptDetails, setGoBackToScriptDetails] = useState(false); // Used for onCancel in delete modal
const selectedScript = useRef<IScript | null>(null);
@@ -86,13 +84,7 @@ const Scripts = ({ router, currentPage, teamIdForApi }: IScriptsProps) => {
const onClickScript = (script: IScript) => {
selectedScript.current = script;
setShowScriptDetailsModal(true);
};
const onCancelScriptDetails = () => {
selectedScript.current = null;
setShowScriptDetailsModal(false);
setGoBackToScriptDetails(false);
setShowEditScriptModal(true);
};
const onEditScript = (script: IScript) => {
@@ -112,12 +104,7 @@ const Scripts = ({ router, currentPage, teamIdForApi }: IScriptsProps) => {
const onCancelDelete = () => {
setShowDeleteScriptModal(false);
if (goBackToScriptDetails) {
setShowScriptDetailsModal(true);
} else {
selectedScript.current = null;
}
selectedScript.current = null;
};
const onDeleteScript = () => {
@@ -200,21 +187,6 @@ const Scripts = ({ router, currentPage, teamIdForApi }: IScriptsProps) => {
onDone={onDeleteScript}
/>
)}
{showScriptDetailsModal && selectedScript.current && (
<ScriptDetailsModal
selectedScriptDetails={{
script_id: selectedScript.current?.id,
name: selectedScript.current?.name,
}}
onCancel={onCancelScriptDetails}
onDelete={() => {
setShowScriptDetailsModal(false);
setShowDeleteScriptModal(true);
setGoBackToScriptDetails(true);
}}
runScriptHelpText
/>
)}
{showEditScripsModal && selectedScript.current && (
<EditScriptModal
scriptId={selectedScript.current.id}
@@ -106,11 +106,12 @@ const EditScriptModal = ({
<>
<form onSubmit={onSubmit}>
<Editor
value={scriptFormData}
onChange={onChange}
isFormField
error={formError}
isFormField
label="Script"
onBlur={onBlur}
onChange={onChange}
value={scriptFormData}
/>
<div className="form-field__help-text">
To run this script on a host, go to the{" "}