-
Notifications
You must be signed in to change notification settings - Fork 3
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
refactor: upgrade Next.js to v15 #279
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…nclude environment variable
avoid unnecesary sourcemaps to get uploaded to sentry
@aws-sdk/client-rekognition, @sentry/nextjs, commitlint, typescript-eslint, and vitest
d113030
to
b1521cf
Compare
marluanespiritusanto
approved these changes
Feb 4, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This pull request includes several changes to improve the codebase, update dependencies, and fix bugs. The most important changes include updating imports, improving Sentry configuration, and making various bug fixes.
Import and Configuration Updates:
instrumentation.ts
: Changedrequire
toimport
forversion
andname
frompackage.json
. Addedenvironment
to Sentry configuration.next.config.ts
: Added Sentry configuration and webpack customizations for better handling of AWS SDK and other packages.Dependency Updates:
package.json
: Updated multiple dependencies and devDependencies to their latest versions.Bug Fixes:
src/actions/cookie.action.ts
: Fixed asynchronous handling ofcookies()
insetCookie
andgetCookie
functions. [1] [2]src/app/[lang]/[...not-found]/page.tsx
: Changedparams
type toPromise
and updated usage accordingly. (src/app/[lang]/[...not-found]/page.tsxL12-R16, src/app/[lang]/[...not-found]/page.tsxL46-R47)src/app/[lang]/account-created/page.tsx
: Changedparams
type toPromise
and updated usage accordingly. (src/app/[lang]/account-created/page.tsxL18-R22, src/app/[lang]/account-created/page.tsxL40-R40)These changes enhance the stability and maintainability of the codebase by ensuring proper import usage, updating dependencies to their latest versions, and fixing bugs related to asynchronous operations.This pull request includes multiple changes to improve the codebase, update dependencies, and fix various issues. The most important changes include refactoring import statements, updating the
next.config.ts
file to include Sentry configuration, and updating dependencies inpackage.json
.Refactoring and Configuration Updates:
instrumentation.ts
to use ES6 import syntax.next.config.ts
, including settings for source map uploading, hiding source maps, and disabling the logger.Dependency Updates:
package.json
, including@aws-amplify
,@emotion
,@mui
,@sentry/nextjs
,next
,react
, andtypescript
.Code Fixes and Improvements:
setCookie
andgetCookie
functions insrc/actions/cookie.action.ts
to await thecookies
call. [1] [2]params
in several files to bePromise<{ lang: Locale }>
, ensuring proper handling of asynchronous operations. (src/app/[lang]/[...not-found]/page.tsxL12-R16, src/app/[lang]/account-created/page.tsxL18-R22, src/app/[lang]/confirmation/page.tsxL10-R19, src/app/[lang]/identification/page.tsxL13-R19, src/app/[lang]/layout.tsxL34-R43, src/app/[lang]/liveness/page.tsxL15-R21, src/app/[lang]/redirection/page.tsxL14-R23)Code Cleanup:
useFormState
withuseActionState
in several form components to improve state management. (src/app/[lang]/confirmation/form.tsxL33-R32, src/app/[lang]/identification/form.tsxL39-R38, src/app/[lang]/register/form.tsxL53-R52)