-
Notifications
You must be signed in to change notification settings - Fork 374
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
feat: Dynamic imports BadWordsNext Chat #598
Conversation
Someone is attempting to deploy a commit to a Personal Account owned by @Dun-sin on Vercel. @Dun-sin first needs to authorize it. |
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.
Looks good, just what I talked about on formatting, and on the other breaking of the components, feel free to create issues when you've thought of ideas for it
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.
Alright, looks good
You did great, thanks for contributing, I hope you will stick around and continue to contribute to this project.
Consider giving this project a star and joining the community discord server if you haven't for more resources and opportunities to connect with others. ππ½hereππ½
The latest updates on your projects. Learn more about Vercel for Git βοΈ
|
Fixes Issue
My PR closes #593
π¨βπ» Changes proposed(What did you do ?)
I created a function called getBadwords, which imports BadWordsNext and EN only when handleSubmit is triggered. Additionally, I introduced a new component called BadwordHideShow, as it also utilizes badword and needed to be externalized.
βοΈ Check List (Check all the applicable boxes)
Note to reviewers
This already enhances performance to some extent. However, I believe we can further modularize the chat section, enabling us to identify more performance optimization opportunities.
π· Screenshots