engineering/tpmjs
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 2.1k

fix: add error handling for collection env vars save

Browse source
This commit is contained in:
Ajax Davis 2026-01-15 00:01:05 +10:00
parent 69ae0dda2f
commit 66670bc9fb
1 changed files with 17 additions and 7 deletions
Download patch file Download diff file Expand all files Collapse all files

24
apps/web/src/app/dashboard/collections/[id]/page.tsx
View file

@ -624,14 +624,24 @@ export default function CollectionDetailPage(): React.ReactElement {
<div className="bg-surface border border-border rounded-lg p-6">
<EnvVarsEditor
value={envVars}
onChange={(newEnvVars) => {
onChange={async (newEnvVars) => {
setEnvVars(newEnvVars);
// Auto-save env vars
fetch(`/api/collections/${collectionId}`, {
method: 'PATCH',
headers: { 'Content-Type': 'application/json' },
body: JSON.stringify({ envVars: newEnvVars }),
});
// Auto-save env vars with error handling
try {
const response = await fetch(`/api/collections/${collectionId}`, {
method: 'PATCH',
headers: { 'Content-Type': 'application/json' },
body: JSON.stringify({ envVars: newEnvVars }),
});
const result = await response.json();
if (!result.success) {
console.error('Failed to save env vars:', result.error);
alert(`Failed to save environment variables: ${result.error || 'Unknown error'}`);
}
} catch (err) {
console.error('Failed to save env vars:', err);
alert('Failed to save environment variables. Please try again.');
}
}}
title="Environment Variables"
description="These variables are passed to tools at runtime. Use them to store API keys and configuration."