-
Notifications
You must be signed in to change notification settings - Fork 0
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
change to use pnpm #45 #46
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
}; | ||
|
||
export const TextEditor: FC<PropTypes> = ({ value, onSave }) => { | ||
export const TextEditor: FC<PropTypes> = ({ value, onChange }) => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Anh khuyến khích là dùng onSubmit
hay onSave
hoặc cái gì đó tương đương, onChange
sẽ dễ bị hiểu lầm là event này fired với mọi ký tự (như cách mà input's onChange hoạt động.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Em đang chuyển qua onChange đúng với ngữ cảnh của nó á anh, em tính chuyển nút save ra bên ngoài! em dùng onSave nhưng khi bấm onSave của thằng tinymce thì nó ko cập nhật đc state hiện tại bên ngoài vào trong func!
package.json
Outdated
@@ -35,7 +35,8 @@ | |||
"react-dom": "18.2.0", | |||
"react-icons": "4.10.1", | |||
"react-virtuoso": "4.5.0", | |||
"swr": "2.2.1" | |||
"swr": "2.2.1", | |||
"tinymce": "^6.6.2" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Install nhớ dùng -E để không có cái tidal ^
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sr anh! em lại quên :/
@@ -39,7 +39,7 @@ CREATE TABLE "User" ( | |||
-- CreateTable | |||
CREATE TABLE "Post" ( | |||
"id" TEXT NOT NULL, | |||
"title": TEXT, | |||
"title" TEXT, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ủa tại sao lại sửa cái này???
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
file migrations được tạo ra tự động từ prisma, và trong 99,99% trường hợp mình không sửa nó bằng tay, vì sau này nó rất dễ conflict với tables trong database
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cái title: TEXT em viết sai cú pháp nên em sửa lại á! Mà chắc em xoá cái đó cho chắc! hôm bữa em thêm vô thử xem nó chạy thế nào ^ ^!
…tjs-template into 45-change-to-use-pnpm
No description provided.