Fix amendment: no version bump on edit, stays on current version
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
@@ -93,12 +93,10 @@ export default function RequestDetail() {
|
||||
setSaving(true);
|
||||
|
||||
if (action === 'edit') {
|
||||
const newVersion = (task.current_version || 0) + 1;
|
||||
await supabase.from('tasks').update({ current_version: newVersion }).eq('id', id);
|
||||
|
||||
// No version bump — amendment notes attach to the current version
|
||||
const { data: newSub } = await supabase.from('submissions').insert({
|
||||
task_id: id,
|
||||
version_number: newVersion + 1,
|
||||
version_number: (task.current_version || 0) + 1,
|
||||
type: 'amendment',
|
||||
service_type: task.title,
|
||||
deadline: revisionForm.deadline || null,
|
||||
@@ -118,8 +116,6 @@ export default function RequestDetail() {
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
setTask(t => ({ ...t, current_version: newVersion }));
|
||||
} else {
|
||||
const newVersion = (task.current_version || 0) + 1;
|
||||
await supabase.from('tasks').update({ status: 'not_started', current_version: newVersion }).eq('id', id);
|
||||
@@ -188,7 +184,7 @@ export default function RequestDetail() {
|
||||
const titleWithVersion = `${task.title} ${vLabel(task.current_version)}`;
|
||||
|
||||
const formTitle = action === 'edit'
|
||||
? `Edit Request — will become ${vLabel((task.current_version || 0) + 1)}`
|
||||
? `Amend Request — ${vLabel(task.current_version || 0)}`
|
||||
: action === 'reopen'
|
||||
? `Request New Revision — will become ${vLabel((task.current_version || 0) + 1)}`
|
||||
: `Request a Revision — will become ${vLabel((task.current_version || 0) + 1)}`;
|
||||
@@ -320,7 +316,9 @@ export default function RequestDetail() {
|
||||
)}
|
||||
<div className="version-header">
|
||||
<div style={{ display: 'flex', alignItems: 'center', gap: 10 }}>
|
||||
{sub.type !== 'amendment' && (
|
||||
<div className="version-number">{vLabel(sub.version_number - 1)}</div>
|
||||
)}
|
||||
<StatusBadge status={sub.type} />
|
||||
</div>
|
||||
<div style={{ fontSize: 12, color: 'var(--text-secondary)' }}>
|
||||
|
||||
Reference in New Issue
Block a user