-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Implement useReducer for textAreaValues #10867
Implement useReducer for textAreaValues #10867
Conversation
// the new comment form comes with an edit form, its toggle state needs to be created as well | ||
// TODO: create multicase for "CREATE NEW COMMENT FORM" in reducers.js |
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.
TODO found
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.
Seems inapplicable because this is within a comment! Possible to skip?
Code Climate has analyzed commit bea891d and detected 1 issue on this pull request. Here's the issue category breakdown:
View more on Code Climate. |
This pull request generated screenshots of many common pages in the running app. You should be able to download and view them here: |
Hi @noi5e its great to hear from you! 🎉 This looks wonderful and many thanks. |
🎉 🎉 🎉 |
Thanks @jywarren! |
Hi again everyone! 👋🏽 I'd like to spend a couple of weeks finishing up a solid React rewrite of the commenting system! Toward the end, I'm going to write a bunch of planning issues so that other contributors can continue the work as needed.
Picking up from where I left off, I'm trying to simplify the app's
state
by using React'suseReducer
feature.(This issue is part of the larger Comment Editor Overhaul Project with Outreachy. Refer to Planning Issues #9069 and #9365 for more context)