Skip to content

Conversation

@adrianjm-dotCMS
Copy link
Contributor

@adrianjm-dotCMS adrianjm-dotCMS commented Jan 15, 2026

Description

This PR completes the migration of the Analytics portlet UI to PrimeNG 21 and Angular 21 standalone components, ensuring visual consistency and modern UX patterns across the DotCMS admin interface.

Testing

  • All existing tests pass
  • Verified component states: loading, error, empty, and data display
  • No console errors in dev mode

Checklist

  • Code follows project style guidelines
  • Tests added/updated where applicable
  • No new console warnings or errors
  • No functional regressions

Visual Impact

  • Dashboard cards now use consistent PrimeNG styling
  • Tables display with proper spacing and typography
  • State messages (loading/error/empty) properly aligned
Screen.Recording.2026-01-15.at.9.49.56.AM.mov

This PR fixes: #34253

This PR fixes: #34253

Copy link
Contributor

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 PR migrates the analytics portlet from SCSS-based styling to Tailwind CSS classes, upgrades to PrimeNG 21, and integrates analytics into the Next.js example. The changes represent a significant frontend modernization effort.

Changes:

  • Converted SCSS stylesheets to inline Tailwind CSS classes across all analytics portlet components
  • Updated PrimeNG component imports and usage patterns for version 21 compatibility
  • Added browser API mocks (MutationObserver, IntersectionObserver, ResizeObserver) for PrimeNG 21 testing requirements
  • Integrated @dotcms/analytics package into the Next.js example with DotContentAnalytics component

Reviewed changes

Copilot reviewed 46 out of 47 changed files in this pull request and generated 19 comments.

Show a summary per file
File Description
examples/nextjs/src/app/layout.js Adds DotContentAnalytics component and imports missing config file
examples/nextjs/package.json Adds @dotcms/analytics dependency
core-web/libs/portlets/dot-analytics/portlet/tsconfig.spec.json Updates module type and adds isolatedModules for PrimeNG 21
core-web/libs/portlets/dot-analytics/portlet/src/test-setup.ts Adds browser API mocks for PrimeNG 21 compatibility
core-web/libs/portlets/dot-analytics/portlet/src/lib/dot-analytics-search/* Replaces SCSS with Tailwind classes in component host and template
core-web/libs/portlets/dot-analytics/portlet/src/lib/dot-analytics-error/* Removes SCSS file and applies Tailwind classes inline
core-web/libs/portlets/dot-analytics/portlet/src/lib/dot-analytics-dashboard/* Migrates all dashboard components from SCSS to Tailwind CSS
Multiple component spec files Updates test selectors and mock setup for PrimeNG 21 changes

@semgrep-code-dotcms-test
Copy link

Legal Risk

The following dependencies were released under a license that
has been flagged by your organization for consideration.

Recommendation

While merging is not directly blocked, it's best to pause and consider what it means to use this license before continuing. If you are unsure, reach out to your security team or Semgrep admin to address this issue.

GPL-2.0

MPL-2.0

@adrianjm-dotCMS adrianjm-dotCMS changed the title migrate analytics portlet Migrate analytics portlet Jan 15, 2026
@semgrep-code-dotcms-test
Copy link

Legal Risk

The following dependencies were released under a license that
has been flagged by your organization for consideration.

Recommendation

While merging is not directly blocked, it's best to pause and consider what it means to use this license before continuing. If you are unsure, reach out to your security team or Semgrep admin to address this issue.

MPL-2.0

@hmoreras hmoreras merged commit 9b98295 into issue-33882-primeng-update Jan 15, 2026
6 checks passed
@hmoreras hmoreras deleted the 34253-migrate-analytics-portlet branch January 15, 2026 17:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

3 participants