Skip to content
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

Translating API Reference -> ReactDOM #31

Closed
wants to merge 4 commits into from

Conversation

charBap
Copy link

@charBap charBap commented Apr 17, 2019

Translating API Reference -> ReactDOM

@netlify
Copy link

netlify bot commented Apr 17, 2019

Deploy preview for hi-reactjs ready!

Built with commit 40cdfa3

https://deploy-preview-31--hi-reactjs.netlify.com

@arshadkazmi42 arshadkazmi42 self-requested a review April 17, 2019 10:23
@arshadkazmi42
Copy link
Member

@charBap You have a pending PR open at #26
Let's close that first then we can start with the review of this PR.
@saranshkataria has left you a couple of feedbacks for that PR.
Can you resolve those, once that is merged I will start reviewing this PR.

@saranshkataria
Copy link
Member

@charBap please let us know if you will be able to complete the other PR so that we can work on this one

@arshadkazmi42 arshadkazmi42 added the 1st Review First phase of review label Jan 24, 2020
@arshadkazmi42 arshadkazmi42 removed the 1st Review First phase of review label Apr 13, 2020
@arshadkazmi42
Copy link
Member

Closing due to inactivity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants