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

Fix PythonPlayGround Error : UNSAFE_componentWillMount #125

Open
wants to merge 15 commits into
base: main
Choose a base branch
from

Conversation

samrat2k03
Copy link
Contributor

what i changed :

  • before i used react-helmet, it is old package ,the maintainer isnt active for 3 years.
  • i moved to react-helmet-async package to fix that kind of error
  • now it has been fixed 👍

package install :

yarn add react-helmet-async

@samrat2k03
Copy link
Contributor Author

video i added :
https://github.com/thecuriousteam/PyLibLog/assets/77772897/061cb94b-57f7-4097-896f-9700de5195b0

just click the link to watch the video ⬆️

@thecuriousteam
Copy link
Owner

There is merge conflict in your pull request, please can you take the latest pull from the main and update us

Thank You

Instructions

  1. Yes you can start working on it by taking latest pull from main branch
  2. Don't Forget to join onboarding group for knowledge flow and direct conversation and support from other developers link Curious Developers Community - Hacktoberfest
  3. Please attach the screenshots and video of the development work once it is done
  4. Contributions more than 3 will be awarded a certificate of appreciation from curious ecosystem please mail to ([email protected]) for claiming certificate and please attach your issue number
  5. Respect Co Developers And Please Reachout to admins if any issues

@samrat2k03
Copy link
Contributor Author

There is merge conflict in your pull request, please can you take the latest pull from the main and update us

Thank You

Instructions

  1. Yes you can start working on it by taking latest pull from main branch
  2. Don't Forget to join onboarding group for knowledge flow and direct conversation and support from other developers link Curious Developers Community - Hacktoberfest
  3. Please attach the screenshots and video of the development work once it is done
  4. Contributions more than 3 will be awarded a certificate of appreciation from curious ecosystem please mail to ([email protected]) for claiming certificate and please attach your issue number
  5. Respect Co Developers And Please Reachout to admins if any issues

I've fixed the conflicts...

@samrat2k03
Copy link
Contributor Author

Mate , make asap fast merge the PR , again it shows the merge conflicts x2 time. Please merge it

@thecuriousteam
Copy link
Owner

Hi @samrat2k03

There is issue in code, Please find the attached screenshot

image

@samrat2k03
Copy link
Contributor Author

Just comment that line , because it already imported on line 2
IMG_20231016_192605.jpg

@samrat2k03
Copy link
Contributor Author

samrat2k03 commented Oct 16, 2023

See the line 2 & line 7 are same, so do comment that line or remove it any one

Please let me know if the issue is resolved or not

@thecuriousteam
Copy link
Owner

@samrat2k03 can you remove one line and initiate for the merging activity

@samrat2k03
Copy link
Contributor Author

Ok

@samrat2k03
Copy link
Contributor Author

I've fixed the issue ,
Screenshot (126)
Screenshot (127)

@samrat2k03
Copy link
Contributor Author

Please let me know, its works on your machine!!!

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.

2 participants