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
Fixed emoji-render-issue using react-easy-emoji #84
Conversation
Get latest Commits.
@kartik918 and @saadpasta Is there anything else that can be added to this PR. |
Can push changes on some domain |
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.
A good PR, just address these changes and show a testing deployment build and this is good to go!
Also @saadpasta , let's squash every PR from now on before merging to avoid too many unnecessary commits on master branch. |
The site is live @ https://mknaveen.netlify.com/ |
There is no image icon showing with the emoji please fix that |
@saadpasta There was a bug in the library I used for adding SVG. So, for now, I have converted it to PNG and it will soon be deployed. |
Is the test build deployed now? |
Yeah, it is deployed on https://mknaveen.netlify.com/. |
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.
Everything looks great, just one last task before this gets merged.
Check the review comment and also you forgot to update the footer ":heart:" icon.
Thanks!
Good job Naveen! |
* Added emoji render lib * Added react-easy-emoji Co-authored-by: Kartik Choudhary <kartikch918@gmail.com>
Fixes #3
I have fixed emoji render issue. From now on if the user needs to enter any emoji in
protofolio.js
, the user should pass it as a parameter inemoji()
. This solves the issue as it converts the emoji to an image so there is no problem.