Skip to content

Code quality and minor UI fixes across schedule, speakers and report views - #5

Merged
fredpena merged 25 commits into
mainfrom
dev
Apr 1, 2026
Merged

fredpena merged 25 commits into
mainfrom
dev

Conversation

@fredpena

@fredpena fredpena commented Apr 1, 2026

Copy link
Copy Markdown
Member

No description provided.

- Set `ZoneId` based on event's timezone string
- Use `timeZone` for current date and time retrieval
- Correct room sorting to handle nulls properly
Define MCP servers for Vaadin and Playwright in new configuration file.
… security config to use SecurityFilterChain; enhance Minio configuration with bucket creation; update application.yml with storage and Hibernate validation; modify Application class to include Lumo styles; update dependencies to newer versions; add Minio Docker setup with healthcheck; improve GitHubService error handling.
- Changed many-to-one and one-to-many relationships to LAZY fetch to improve performance
- Added @EntityGraph annotations to repositories for efficient eager loading of related entities
- Updated Minio configuration to include a primary bean and a qualified publicMinioClient bean
- Modified ScheduleView and related classes to inject and use the new speakerService for hydrated data
- Enhanced SpeakerRepository with @EntityGraph for sessions, room, and tags
- Updated Session and SessionRating entities with @NamedEntityGraph for display associations
- Adjusted application.yml for correct public-endpoint configuration
- Changed popover role from "overlayRole" to "menu" for accessibility
- Added "Change password" item to user menu
- Adjusted popover content order for better UX
- Updated primary and warning colors in CSS for improved theme consistency
Refactor image handling across views and services to use object storage with photoKey. Adds storage deletion on entity removal and updates image URL retrieval. Includes SQL migration for new photoKey columns and drops old bytea columns.
…dencies

- Marked ScheduleViewDetails and SpeakersViewDetails with @uiscope and @component for Spring management
- Injected these components into ScheduleView, FavoriteView, SpeakersView, SpeakersViewDetails, and ScheduleViewDetails constructors
- Replaced direct instantiation with dependency injection
- Removed redundant object creation in views
- Ensured consistent use of injected components for navigation and interactions
…c improvements

- Redesigned login CSS with a dark overlay, side-by-side layout, and styled brand panel
- Enhanced form card with shadows, rounded corners, and full-width input fields
- Improved header, footer, and back-to-home link styling for clarity and consistency
- Added responsive media queries for tablets and mobile devices, including stacked layout and scaled elements
- Removed background image and simplified background gradients for a cleaner look
- Updated service classes to include `findAll(Specification)` methods for flexible querying.
- Refactored list initialization to use `VirtualList` with custom renderers.
- Improved selection handling by tracking selected items and refreshing data provider.
- Enhanced UI components with consistent styling and layout adjustments.
- Minor code cleanup and renaming for clarity.
…ce Mailhog with Mailpit

- Bump postgres image 15 → 18 in both docker-compose files
- Update volume mount to /var/lib/postgresql (required by pg18 data layout)
- Replace mailhog with Mailpit and add SMTP auth env vars
- Replace MinIO with SeaweedFS in docker-compose.yml
- Harden Dockerfile: non-root user, WORKDIR, JVM container flags
…ps and PostgreSQL 18

- Replace curl one-liner with apt repository method from docs.docker.com/engine/install/debian
- Update PostgreSQL version reference to 18 in getting-started
…h stripping in Postal service

- Use RestClient.builder() static factory to avoid false autowiring warning
- Replace stripTrailing("/") with replaceAll("/+$", "") — stripTrailing takes no arguments
- Remove redundant catch-rethrow block for EmailSendException
…, show item labels

- Disable openOnClick so popover never opens on click
- Replace SideNavItem with plain Div in popover to avoid font-size:0 from collapsed CSS
- Navigation via UI.navigate() preserves SPA behavior
- Clean up obsolete vaadin-side-nav-item popover CSS rules
… and report views

- Mark non-serializable service fields as transient (ReportBugView)
- Use unnamed exception variable _ for unused catch param (ReportBugView)
- Fix RateView page title from Spanish to English
- Refactor ScheduleView, SpeakersView, VCardView for cleaner layout and component reuse
@fredpena
fredpena merged commit ea2245f into main Apr 1, 2026
0 of 2 checks passed
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.

1 participant