fix: handle target field being empty and translate to null

Author: oxcened

src/app/dashboard/cash-flow/envelopes/config.tsx

@@ -114,6 +114,7 @@ export const envelopeColumns = [
           </div>
 
           <Progress
+            key={progressValue.toNumber()}
             value={progressValue.toNumber()}
             className={cn(
               "max-w-[300px]",

src/server/api/routers/envelope.ts

@@ -70,11 +70,7 @@ export const envelopeRouter = createTRPCRouter({
     .mutation(async ({ input, ctx }) => {
       const updated = await ctx.db.envelope.update({
         where: { id: input.id, createdById: ctx.session.user.id },
-        data: {
-          ...(input.target ? { target: input.target } : {}),
-          name: input.name,
-          amount: input.amount,
-        },
+        data: input,
       });
 
       return updated;

src/trpc/schemas/envelope.ts

@@ -2,7 +2,12 @@ import * as yup from "yup";
 
 export const createEnvelopeSchema = yup.object({
   name: yup.string().label("Name").required(),
-  target: yup.number().label("Target").optional(),
+  target: yup
+    .number()
+    .label("Target")
+    .optional()
+    .nullable()
+    .transform((value) => (isNaN(value) ? null : value)),
   amount: yup.number().label("Amount").required(),
   priority: yup.number().label("Priority").required(),
 });