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

refactor: upgrade app and it's dependencies to support next.js 14 #237

Merged
merged 46 commits into from
Jul 24, 2024

Conversation

JeffreyArt1
Copy link
Contributor

@JeffreyArt1 JeffreyArt1 commented Jun 17, 2024

Sentry

  • Add instrumentation.ts for Sentry initialization
  • Update sentry.client.config.ts to use Sentry.replayIntegration
  • Remove sentry.edge.config.ts and sentry.server.config.ts
  • Add global error handling with Sentry in src/app/global-error.tsx

Tests

  • Move tests/app.test.ts to __tests__/app.test.ts
  • Add new test __tests__/page.test.tsx

Liveness

  • Add amplify-server.ts helper for server-side Amplify configuration
  • Refactor rekognition.ts to use runWithAmplifyServerContext
  • Refactor LivenessModal component to remove unnecessary padding
  • Refactor LivenessQuickStart component to disable start screen and use new configuration
  • Update Dockerfile to use amplifyconfiguration.json instead of aws-exports.js

Misc

  • Add robots.txt to disallow all user agents
  • Add common URLs to src/common/index.ts
  • Update report schema in report.schema.ts to include cedula validation
  • Upgrade various dependencies in package.json
  • Update getCookie function in src/actions/cookie.action.ts to parse JSON correctly
  • Update findIamCitizen function in src/actions/iam.action.ts to use @ory/client API
  • Update next.config.js to support Next.js 14 features
  • Use next.js GoogleTagManager component

This closes #179

@marluanespiritusanto marluanespiritusanto merged commit 18c1611 into master Jul 24, 2024
7 checks passed
@marluanespiritusanto marluanespiritusanto deleted the refactor/next-14 branch July 24, 2024 12:47
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.

Migración a nextjs 14
2 participants