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

feat: Updating Next JS to a newer version #26

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

krlz16
Copy link
Collaborator

@krlz16 krlz16 commented Oct 23, 2024

Updating Nextjs

Next has been updated to be able to use its latest features and protect the routes and its server.

  • The API_URL was removed since it is not necessary to access the server through it. Instead, we use 'server actions'
  • Files were moved inside the src folder.
  • In the middleware file the route is protected so that no other route can be accessed, only the main route '/'
  • The functionality remains the same, nothing has changed in its logic.
  • All test-related files have been removed, they have not been tested at any time and tests need to be added in a next iteration.

	Next has been updated to be able to use its latest
	features and protect the routes and its server
@krlz16 krlz16 requested a review from ivegabr October 23, 2024 01:09
Copy link

@ivegabr ivegabr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok

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.

2 participants