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 |
arshadkazmi42
left a comment
arshadkazmi42
left a comment
There was a problem hiding this comment.
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 changesDetails📦 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! 🙌 |
arshadkazmi42
left a comment
arshadkazmi42
left a comment
There was a problem hiding this comment.
Added more comments till line 212.
Lets get these fixed and continue further
review changes
arshadkazmi42
left a comment
arshadkazmi42
left a comment
There was a problem hiding this comment.
added more till lines 313
review changes till line number 313
arshadkazmi42
left a comment
arshadkazmi42
left a comment
There was a problem hiding this comment.
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 👍
arshadkazmi42
left a comment
arshadkazmi42
left a comment
There was a problem hiding this comment.
Great work @PriyankaBelhekar77
Thank you for making all the changes.
PR looks good now and we can get this merged.
2 participants
I have translated the State as a Snapshot page. Please review it