Merge pull request #74 from afadil/dev

Fix withdrawal, and deposit form validation

package.json

@@ -1,7 +1,7 @@
 {
   "name": "wealthfolio-app",
   "private": true,
-  "version": "1.0.8",
+  "version": "1.0.9",
   "type": "module",
   "scripts": {
     "dev": "vite",

src-tauri/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "wealthfolio-app"
-version = "1.0.8"
+version = "1.0.9"
 description = "Portfolio tracker"
 authors = ["Aziz Fadil"]
 license = "LGPL-3.0"

src-tauri/tauri.conf.json

@@ -8,7 +8,7 @@
   },
   "package": {
     "productName": "Wealthfolio",
-    "version": "1.0.8"
+    "version": "1.0.9"
   },
   "tauri": {
     "allowlist": {

src/lib/schemas.ts

@@ -34,7 +34,7 @@ export const newGoalSchema = z.object({
 
 export const newActivitySchema = z.object({
   id: z.string().uuid().optional(),
-  accountId: z.string().min(1, { message: 'Account ID is required' }),
+  accountId: z.string().min(1, { message: 'Please select an account.' }),
   activityDate: z.date(),
   currency: z.string().min(1, { message: 'Currency is required' }),
   fee: z.coerce
@@ -52,21 +52,28 @@ export const newActivitySchema = z.object({
     })
     .positive({ message: 'Quantity must be a positive number.' }),
   assetId: z.string().min(1, { message: 'Asset ID is required' }),
-  activityType: z.enum([
-    'BUY',
-    'SELL',
-    'DIVIDEND',
-    'INTEREST',
-    'DEPOSIT',
-    'WITHDRAWAL',
-    'TRANSFER_IN',
-    'TRANSFER_OUT',
-    'CONVERSION_IN',
-    'CONVERSION_OUT',
-    'FEE',
-    'TAX',
-    'SPLIT',
-  ]),
+  activityType: z.enum(
+    [
+      'BUY',
+      'SELL',
+      'DIVIDEND',
+      'INTEREST',
+      'DEPOSIT',
+      'WITHDRAWAL',
+      'TRANSFER_IN',
+      'TRANSFER_OUT',
+      'CONVERSION_IN',
+      'CONVERSION_OUT',
+      'FEE',
+      'TAX',
+      'SPLIT',
+    ],
+    {
+      errorMap: () => {
+        return { message: 'Please select an activity type.' };
+      },
+    },
+  ),
   unitPrice: z.coerce
     .number({
       required_error: 'Please enter a valid price.',

src/pages/activity/components/activity-form.tsx

@@ -129,6 +129,8 @@ export function ActivityForm({ accounts, defaultValues, onSuccess = () => {} }:
     form.setValue('currency', currentAccountCurrency);
     if (CASH_ACTIVITY_TYPES.includes(watchedType)) {
       form.setValue('assetId', `$CASH-${currentAccountCurrency}`);
+      form.setValue('unitPrice', 1);
+      form.setValue('fee', 0);
     }
   }, [currentAccountCurrency, watchedType]);
 
@@ -264,7 +266,6 @@ export function ActivityForm({ accounts, defaultValues, onSuccess = () => {} }:
     </Form>
   );
 }
-
 interface CashActivityFieldsProps {
   currentAccountCurrency: string;
 }