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

Jwt project updated #149

Open
wants to merge 5 commits into
base: jwt
Choose a base branch
from
Open

Jwt project updated #149

wants to merge 5 commits into from

Conversation

FabricioMoreno
Copy link

Hello! I hope you're having a good day. I would like to propose this pull request for you to review the changes I've made. I've fixed some bugs that I found in the code and added some enhancements that I believe may be useful. Here are the details of the changes I've made:

I updated some outdated packages, which should help improve the project's stability and security.

I added a base URL in Axios, since all routing in the api directory starts with /api. This should simplify the Axios configuration and make the code easier to maintain.

I changed the import of app.css to App.css. This should help maintain consistency in the project and avoid confusion.

I added the proxy property in package.json to http://localhost:5000/. This should help properly configure the server for HTTP requests and avoid CORS problems.

I also installed the cors package to fix CORS issues on the server. Adding this package should allow HTTP requests to be made correctly and avoid CORS errors in the future.

I hope you find these changes useful and are happy to merge them into the project. If you have any questions or additional comments, please let me know. Thank you again!

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

Successfully merging this pull request may close these issues.

1 participant