-
-
Notifications
You must be signed in to change notification settings - Fork 166
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
Convert React component to functional component #1057
Comments
@alexanmtz please assign me this issue👨💻 |
Hey @Asin-Junior-Honore, we have this PR: #1058; you can continue from there; please check it out and test locally. You need to open the user skill page: https://gitpay.me/#/profile/user-account/skills Let me know if you need any help |
@alexanmtz when i test the site in my Browser locally it displays the landing page very well and clearly but when i try to login i get this message👇 although my internet is working fine |
@Asin-Junior-Honore, the local integration to log in via Github is not enabled by default, so please create an account with an e-mail and a password. The activation to activate your account locally is usually from e-mail, but locally, there's no mail server, so you will be able to activate your account by copying the activation link on your local server log. |
@alexanmtz i am really sorry for the late response but could you please give more context on the required solution please🙏 |
@Asin-Junior-Honore register with a regular account using e-mail instead of Github locally, then go to your terminal when you ran the server and you will see the email content for account activation there, then you can copy and paste the link, then go to your skills section on: http://localhost:8082/#/profile/user-account/skills locally and you should open the page with any errors :-) |
@alexanmtz I followed your instructions but am still not getting any activation details in my terminal besides I am always getting error in my terminal whenever I try npm I |
Which error are you facing? Have you followed the documentation on README
to make the proper setup?
…--
Alexandre Magno Teles Zimerer
*Software Engineer*
https://alexandremagno.net/
github.com/alexanmtz
@alexanmtz
On Wed, 21 Feb 2024 at 00.56, Asin-Honore ***@***.***> wrote:
@alexanmtz <https://github.com/alexanmtz> I followed your instructions
but am still not getting any activation details in my terminal besides I am
always getting error in my terminal whenever I try npm I
—
Reply to this email directly, view it on GitHub
<#1057 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAAVWCGLJYMUQMSYD7GI6OTYUUZZLAVCNFSM6AAAAABC6L7GI6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNJVGQYTANJUHA>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
@alexanmtz Can you please assign me to this issue ? |
Yes @Dheeraj4103 , feel free to send a PR with your solution 👌 |
Thanks @alexanmtz , I will start working on it. |
Hey @alexanmtz , there are lot of libraries that are deprecated, even if I use docker to build project both frontend and backend fails. |
@Dheeraj4103 I guess we need to fix docker-compose to make it work properly: And for run the project without docker and install with decrecated dependencies you need to run: |
hey @alexanmtz, some packages require node v12 but some require version > v14, even after using command given by I'm not able to setup project |
@Dheeraj4103 You need to use Node v17 to be able to run the backend server, and for the fron-end you can run the project with |
hey @alexanmtz I tried what you said but it didn't work. |
@Dheeraj4103, have you updated and made it work? If that's the case, you can send a PR with the fix |
@Dheeraj4103 have you tried this? https://stackoverflow.com/questions/65594119/node-sass-usage-is-deprecated-and-will-be-removed-in-a-future-major-version And just to make sure, your issue is on the Node server right? |
Hey @alexanmtz I have updated package.json for backend and it is working fine now, Thanks for help. |
That's a great progress @Dheeraj4103, and it's really weird the seed having this output. Have you ran the migrations successfully with And what's your OS? Windows? Linux? Mac? |
Hey @alexanmtz, |
@Dheeraj4103 please check our README file for running the seed on Windows: |
hey @alexanmtz I have set up project. Should I create a seperate PR for updated package.json |
Hey @alexanmtz, But there are few test cases that are failing, which are related to |
Hey @alexanmtz, |
thats great @Dheeraj4103 , I guess the tests are failing because you need to run the migration for tests too, so have you tried |
|
Yeah @Dheeraj4103 I will start to review your PR :) |
Your PR was merged and already in production @Dheeraj4103 , thanks for the contribution, and feel free to start work in any other issue you're interested |
Change the file './frontend/src/components/profile/skill.js' to a functional react component.
The text was updated successfully, but these errors were encountered: