Skip to content
Snippets Groups Projects
Unverified Commit 48b7d54b authored by Sean Hatfield's avatar Sean Hatfield Committed by GitHub
Browse files

[FIX] Delete SQL agent connection bug fix (#1635)

fix update agent settings button not appearing when deleting sql connection
parent 3434bf46
No related branches found
No related tags found
No related merge requests found
...@@ -9,7 +9,7 @@ export const DB_LOGOS = { ...@@ -9,7 +9,7 @@ export const DB_LOGOS = {
"sql-server": MSSQLLogo, "sql-server": MSSQLLogo,
}; };
export default function DBConnection({ connection, onRemove }) { export default function DBConnection({ connection, onRemove, setHasChanges }) {
const { database_id, engine } = connection; const { database_id, engine } = connection;
function removeConfirmation() { function removeConfirmation() {
if ( if (
...@@ -20,6 +20,7 @@ export default function DBConnection({ connection, onRemove }) { ...@@ -20,6 +20,7 @@ export default function DBConnection({ connection, onRemove }) {
return false; return false;
} }
onRemove(database_id); onRemove(database_id);
setHasChanges(true);
} }
return ( return (
......
...@@ -9,6 +9,7 @@ export default function AgentSQLConnectorSelection({ ...@@ -9,6 +9,7 @@ export default function AgentSQLConnectorSelection({
settings, settings,
toggleSkill, toggleSkill,
enabled = false, enabled = false,
setHasChanges,
}) { }) {
const { isOpen, openModal, closeModal } = useModal(); const { isOpen, openModal, closeModal } = useModal();
const [connections, setConnections] = useState( const [connections, setConnections] = useState(
...@@ -72,6 +73,7 @@ export default function AgentSQLConnectorSelection({ ...@@ -72,6 +73,7 @@ export default function AgentSQLConnectorSelection({
}) })
); );
}} }}
setHasChanges={setHasChanges}
/> />
))} ))}
<button <button
......
...@@ -100,6 +100,7 @@ export default function WorkspaceAgentConfiguration({ workspace }) { ...@@ -100,6 +100,7 @@ export default function WorkspaceAgentConfiguration({ workspace }) {
skills={agentSkills} skills={agentSkills}
toggleAgentSkill={toggleAgentSkill} toggleAgentSkill={toggleAgentSkill}
settings={settings} settings={settings}
setHasChanges={setHasChanges}
/> />
{hasChanges && ( {hasChanges && (
<button <button
...@@ -143,7 +144,7 @@ function LoadingSkeleton() { ...@@ -143,7 +144,7 @@ function LoadingSkeleton() {
); );
} }
function AvailableAgentSkills({ skills, settings, toggleAgentSkill }) { function AvailableAgentSkills({ skills, settings, toggleAgentSkill, setHasChanges }) {
return ( return (
<div> <div>
<div className="flex flex-col mb-8"> <div className="flex flex-col mb-8">
...@@ -211,6 +212,7 @@ function AvailableAgentSkills({ skills, settings, toggleAgentSkill }) { ...@@ -211,6 +212,7 @@ function AvailableAgentSkills({ skills, settings, toggleAgentSkill }) {
settings={settings} settings={settings}
toggleSkill={toggleAgentSkill} toggleSkill={toggleAgentSkill}
enabled={skills.includes("sql-agent")} enabled={skills.includes("sql-agent")}
setHasChanges={setHasChanges}
/> />
</div> </div>
</div> </div>
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment