Skip to content

fix(account-group): fix eslint and type errors in account group - #217

Merged
dimpram101 merged 3 commits into
developmentfrom
fix/account-group-eslint-errors
Jul 28, 2026
Merged

fix(account-group): fix eslint and type errors in account group#217
dimpram101 merged 3 commits into
developmentfrom
fix/account-group-eslint-errors

Conversation

@yusmanata

Copy link
Copy Markdown
Contributor

Memperbaiki error ESLint dan TypeScript pada form Account Group agar sepenuhnya patuh terhadap eslint-local-rules terbaru dan validasi tipe Vue TypeScript.

Perubahan

ESLint & TypeScript Fixes

  • create.vue & edit.vue — Memperbaiki type mismatch pada prop komponen <Textarea>. Mengubah statis string rows="3" menjadi number binding :rows="3" agar sesuai dengan definisi properti komponen.
  • edit.vue — Menambahkan komponen <FieldError> pada field checkbox is_active. Meskipun schema saat ini hanya berupa boolean opsional, penambahan ini diwajibkan oleh struktur statis dari rule fertomulia/vee-field-requires-error-display untuk menjamin form tidak gagal menampilkan error secara sunyi (silent failure).

Checklist

  • ESLint: 0 errors, 0 warnings (npx eslint pada file terdampak)
  • TypeScript build typecheck lolos (tidak ada garis merah pada IDE)
  • Form Create dan Edit tetap berfungsi normal tanpa perubahan visual yang merusak

Closes #216

@yusmanata
yusmanata requested a review from dimpram101 July 23, 2026 06:54
@yusmanata yusmanata self-assigned this Jul 23, 2026
@vercel

vercel Bot commented Jul 23, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
erp-web-frontend Ready Ready Preview, Comment Jul 27, 2026 12:11pm
erp-web-frontend-3xc2 Ready Ready Preview, Comment Jul 27, 2026 12:11pm

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

kenapa di edit pakai data dari list?
kenapa ga dibuat composable untuk ambil satu data aja?

kemudian ganti onMounted jadi watch, karena akan langsung redirect jika merefresh halaman tersebut

@dimpram101 dimpram101 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Komponen Select ganti jadi ShadSelect

@dimpram101
dimpram101 merged commit 87ac284 into development Jul 28, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fix ESLint errors in Account Group

2 participants