Skip to content

Conversation

@gugu
Copy link
Contributor

@gugu gugu commented Jan 13, 2026

No description provided.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request upgrades the Angular frontend application from version 19 to version 20, along with related framework dependencies and configuration updates. The changes include package updates, Sentry SDK migration, Material Design CSS variable updates, and test modernization.

Changes:

  • Upgraded Angular packages from ~19.x to ~20.x
  • Migrated from @sentry/angular-ivy to @sentry/angular (v10.33.0)
  • Updated Material Design CSS variables from --mdc-* to --mat-* prefix
  • Modernized TypeScript configuration and test utilities
  • Updated browser support targets

Reviewed changes

Copilot reviewed 35 out of 37 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
frontend/package.json Angular and related dependencies upgraded to v20, Sentry package updated
frontend/tsconfig.json Module resolution changed to "bundler", formatting standardized
frontend/src/main.ts Sentry API updated to use browserTracingIntegration()
frontend/src/types/globals.d.ts New file declaring Window interface extensions
frontend/src/custom-theme.scss Material CSS variables updated to --mat-* prefix
frontend/src/app/services/*.ts Sentry import paths updated from angular-ivy to angular
frontend/src/app/models/company.ts Import paths updated for Sentry
frontend/src/app/components/**/*.ts Import paths and formatting updates
frontend/src/app/components/**/*.spec.ts TestBed.get() replaced with TestBed.inject()
frontend/src/app/components/**/*.css Material CSS variables updated throughout
frontend/src/app/app.component.ts Sentry imports and Window interface usage updated
frontend/browserslist Simplified to modern browser targeting
frontend/CLAUDE.md Documentation updated to reflect Sentry package change

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

}
// app initialization if user is logged in (session restoration)
else if (expirationToken) {
const expirationTime = expirationToken ? new Date(expirationToken) : null;
Copy link

Copilot AI Jan 13, 2026

Choose a reason for hiding this comment

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

This use of variable 'expirationToken' always evaluates to true.

Suggested change
const expirationTime = expirationToken ? new Date(expirationToken) : null;
const expirationTime = new Date(expirationToken);

Copilot uses AI. Check for mistakes.
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