Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tại sao lại phải generate dvhcvn.json sang data.ts? #100

Open
ooker777 opened this issue Oct 16, 2024 · 3 comments
Open

Tại sao lại phải generate dvhcvn.json sang data.ts? #100

ooker777 opened this issue Oct 16, 2024 · 3 comments

Comments

@ooker777
Copy link

ooker777 commented Oct 16, 2024

API từ npm lấy dữ liệu tại transformers/js-dvhcvn/src/data.ts, không phải data/dvhcvn.json, và có file transformers/js-dvhcvn/bin/generate.ts để làm điều đó. Mình chưa hiểu lắm ý đồ của việc này là gì? Bạn có thể giải thích thêm ko?

@daohoangson
Copy link
Owner

Đóng gói lại để dễ xài hơn với các bundler khác nhau đó bạn. Do không phải cái nào cũng hỗ trợ import JSON trực tiếp.

@ooker777
Copy link
Author

ooker777 commented Nov 1, 2024

Sao lại ko import đc từ json nhỉ? Mình thấy các file không thống nhất dữ liệu cho lắm

@daohoangson
Copy link
Owner

Một số bundler cũ nó không import được JSON hoặc import JSON nhưng mất typing nên khi publish lên NPM thì mình có define lại types + chuyển JSON về script để dễ xài hơn. Trong package thì không giữ data shape ban đầu do nhu cầu khi sử dụng nó khác với nhu cầu khi lưu trữ / truyền dữ liệu.

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

No branches or pull requests

2 participants