Open
Conversation
Co-Authored-By: martty-code <nesalia.inc@gmail.com> Co-Authored-By: Claude Sonnet <noreply@anthropic.com>
Extract version from git tag (e.g., v2.1.3) and update package.json automatically in the release workflow before building the app. Co-Authored-By: martty-code <nesalia.inc@gmail.com> Co-Authored-By: Claude Sonnet <noreply@anthropic.com>
- Enhance database schemas and IPC handlers - Add notes store for employee management - Improve export functionality (CSV, Excel, PDF) - Update UI components with better error handling - Fix various bugs and improve code quality Co-Authored-By: martty-code <nesalia.inc@gmail.com> Co-Authored-By: Claude Sonnet <noreply@anthropic.com>
npm ci uses exact versions from package-lock.json, so the version change in package.json wasn't being picked up. Using npm install allows the version to be properly updated. Co-Authored-By: martty-code <nesalia.inc@gmail.com> Co-Authored-By: Claude Sonnet <noreply@anthropic.com>
The previous version extraction didn't work on Windows runners because $GITHUB_OUTPUT works differently in PowerShell. Using proper PowerShell syntax to write to the output file. Co-Authored-By: martty-code <nesalia.inc@gmail.com> Co-Authored-By: Claude Sonnet <noreply@anthropic.com>
Temporary fix to reduce app startup time from ~20s to ~5s by removing the update check that makes network requests. Co-Authored-By: martty-code <nesalia.inc@gmail.com> Co-Authored-By: Claude Sonnet <noreply@anthropic.com>
- Created AlertsPageTable component with internal useAlerts hook - Removed duplicate filters and state from alerts-page.tsx - Replaced AlertsTable with AlertsPageTable in home-page - Unified filter styling across the application Co-Authored-By: martty-code <nesalia.inc@gmail.com> Co-Authored-By: Claude Sonnet <noreply@anthropic.com>
The conditional rendering was based on parent's data while the component fetches its own data, causing empty state to show incorrectly. Co-Authored-By: martty-code <nesalia.inc@gmail.com> Co-Authored-By: Claude Sonnet <noreply@anthropic.com>
The severity filter was being converted to undefined when "all", causing different query keys than what home-page was using. Now severity passes through "all" value to match home-page behavior. Co-Authored-By: martty-code <nesalia.inc@gmail.com> Co-Authored-By: Claude Sonnet <noreply@anthropic.com>
- Simplified component by removing unused states and filters - Aligned loading skeleton with other pages - Cleaned up imports and structure - Removed duplicate TooltipProvider nesting Co-Authored-By: martty-code <nesalia.inc@gmail.com> Co-Authored-By: Claude Sonnet <noreply@anthropic.com>
Created index.ts that exports Homepage object with KPIS and Table wrappers, then updated home-page to use the new import pattern. Co-Authored-By: martty-code <nesalia.inc@gmail.com> Co-Authored-By: Claude Sonnet <noreply@anthropic.com>
Created @/components/pages/index.tsx with Page.Content component as a simple wrapper div for page content. Co-Authored-By: martty-code <nesalia.inc@gmail.com> Co-Authored-By: Claude Sonnet <noreply@anthropic.com>
Updated home-page to use Page.Content from @/components/pages to wrap KPI and Table components. Co-Authored-By: martty-code <nesalia.inc@gmail.com> Co-Authored-By: Claude Sonnet <noreply@anthropic.com>
Created Page.Root that wraps TooltipProvider and the page structure. Updated home-page to use Page.Root instead of inline TooltipProvider. Co-Authored-By: martty-code <nesalia.inc@gmail.com> Co-Authored-By: Claude Sonnet <noreply@anthropic.com>
Co-Authored-By: martty-code <nesalia.inc@gmail.com> Co-Authored-By: Claude Sonnet <noreply@anthropic.com>
Reverted the severity filter to convert "all" to undefined to match previous behavior and see if that restores the alerts visibility. Co-Authored-By: martty-code <nesalia.inc@gmail.com> Co-Authored-By: Claude Sonnet <noreply@anthropic.com>
The old version was passing filter values even when "all", creating different query keys. Now passing all values through to match. Co-Authored-By: martty-code <nesalia.inc@gmail.com> Co-Authored-By: Claude Sonnet <noreply@anthropic.com>
Added useORPCReady hook to useAlerts to ensure IPC is ready before fetching. This prevents the "IPC not ready, returning empty" issue that was causing alerts to not appear. Co-Authored-By: martty-code <nesalia.inc@gmail.com> Co-Authored-By: Claude Sonnet <noreply@anthropic.com>
Added employee ID column that displays formatted ID (e.g., #1) and links to the employee's detail page. Co-Authored-By: martty-code <nesalia.inc@gmail.com> Co-Authored-By: Claude Sonnet <noreply@anthropic.com>
Following employees-table.tsx pattern, employee ID is now shown as subtext below the employee name in the same cell, with a link to the employee's detail page. Co-Authored-By: martty-code <nesalia.inc@gmail.com> Co-Authored-By: Claude Sonnet <noreply@anthropic.com>
- Added category column showing CACES category or visit type - Added daysLeft column with color-coded urgency badges - Added translations for new columns in EN and FR Co-Authored-By: martty-code <nesalia.inc@gmail.com> Co-Authored-By: Claude Sonnet <noreply@anthropic.com>
Added getCategoryBadge function to display category with a colored dot badge, similar to the type badge. Co-Authored-By: martty-code <nesalia.inc@gmail.com> Co-Authored-By: Claude Sonnet <noreply@anthropic.com>
Reordered table columns to: employee, type, category, severity, daysLeft, date Co-Authored-By: martty-code <nesalia.inc@gmail.com> Co-Authored-By: Claude Sonnet <noreply@anthropic.com>
Changed i18n config to default to French instead of English. Co-Authored-By: martty-code <nesalia.inc@gmail.com> Co-Authored-By: Claude Sonnet <noreply@anthropic.com>
Document WEMS project purpose and core features including employee management, certifications (CACE, medical visits, driving authorizations), training, and document management. Co-Authored-By: martty-code <nesalia.inc@gmail.com> Co-Authored-By: Claude Sonnet <noreply@anthropic.com>
Add @@ alias for ./src/core in addition to @ alias for ./src/renderer/src Co-Authored-By: martty-code <nesalia.inc@gmail.com> Co-Authored-By: Claude Sonnet <noreply@anthropic.com>
- Change npm run make to npm run package in testing.yaml - Fix e2e test title expectation from electron-shadcn to WEMS Co-Authored-By: martty-code <nesalia.inc@gmail.com> Co-Authored-By: Claude Sonnet <noreply@anthropic.com>
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Summary
Test plan
🤖 Generated with Claude Code