-
Notifications
You must be signed in to change notification settings - Fork 128
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
Translate State as a Snapshot #236
Translate State as a Snapshot #236
Conversation
@arshadkazmi42 please review it |
Thank you for working on this. I will get to it asap. |
@arshadkazmi42 please review 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.
Thank you for working on this.
I have added review till line 100.
Lets get these fixed and then we can continue further with the review.
Also, refer to our review guidelines #23
Title translation
Size changes📦 Next.js Bundle Analysis for react-devThis analysis was generated by the Next.js Bundle Analysis action. 🤖 This PR introduced no changes to the JavaScript bundle! 🙌 |
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.
Added more comments till line 212.
Lets get these fixed and continue further
review changes
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.
added more till lines 313
review changes till line number 313
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.
Thank you for making all the changes.
I have added review for rest of the file, once these are done we can get this merged 👍
review changes till line no 432
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.
Great work @PriyankaBelhekar77
Thank you for making all the changes.
PR looks good now and we can get this merged.
I have translated the State as a Snapshot page. Please review it