develop#64
Merged
andrejvysny merged 37 commits intomainfrom Feb 7, 2026
Merged
Conversation
- Added ImportFailureDto to encapsulate details of import failures. - Enhanced ImportFailurePersister to handle SQL failures and persist them. - Introduced ImportFailureStatus and ImportFailureType enums for better categorization of failures. - Created TransactionPersistenceResult to capture results of transaction persistence, including SQL failures. - Updated TransactionPersister to utilize TransactionPersistenceResult for tracking success and failures. - Added migration to include fingerprint and reconciled details in the transactions table. - Optimized TransactionDataParser to generate fingerprints for transactions. - Updated TransactionImportService to process and log SQL failures. - Enhanced frontend components to display fingerprint information in error details. - Documented performance optimizations for the RuleEngine.
…tor; refactor action handling
…e and append options
… for field values
…nd Tag models with getUserId method
… handling in BulkActionMenu
…est as incomplete in RuleEngineTest
…s repository interfaces
Contributor
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the
✨ Finishing touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
…mance and parallel execution
…mance and parallel execution
…er with monthly statistics and transaction handling
…bugging, testing, and validation workflows
…s and improve repository interfaces
…alculations to exclude transfers from income/expense summaries
…evelopment in Spendly project
…ing guidance for enhanced delegation in Spendly project
…nd validation enhancements
… for enhanced data handling
…for enhanced account balance management and analytics
…d financial analysis
… account details, and improved requisition handling
…tion for enhanced transaction management
…detachment features and detailed statistics
…on features for enhanced data integrity
…t functionality for enhanced testing and automation
…ch + field leaking - HeaderAnalyzerTest: processed_date detection, Slovak diacritics, UTF-8 Jaro-Winkler (7 new tests) - TransactionDataParserTest: eu/us/simple format aliases, processed_date field (6 new tests) - ImportCsvCommandTest: processed_date e2e, tie-break, fix pre-existing date-format bug (2 new + 2 fixed) - HeaderAnalyzer: return best match across all fields instead of first match ≥ 0.85 - ImportCsvCommand: add currency to transactionFields, alias date→booked_date, skip unknown fields - E2E plan: fix SK9509 delimiter from ; to , Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…for improved environment setup - Enhanced `compose.prod.yml` with explicit environment variables for production settings. - Removed unnecessary `.env` file dependency and added default values for optional variables. - Simplified Dockerfile by removing Xdebug installation and configuration, focusing on essential PHP extensions. - Updated GitHub Actions workflow to reflect the change from `vite.config.js` to `vite.config.ts`.
- Modified `compose.prod.yml` to include build context and target for production. - Updated Dockerfile to clarify production stage and streamline permissions setup. - Adjusted GitHub Actions workflow to specify the production target for builds.
|
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.
Pull Request
Description
Brief description of the changes made in this PR.
Type of Change
Related Issues
Fixes #(issue number)
Relates to #(issue number)
Changes Made
Detailed Changes
Financial Domain Impact
If this PR affects financial functionality:
Financial Data Accuracy
Testing
Test Coverage
Security Considerations
Breaking Changes
If this PR introduces breaking changes, please describe:
Screenshots/Videos
If applicable, add screenshots or videos to demonstrate the changes.
Before:
[Screenshot/Video of before state]
After:
[Screenshot/Video of after state]
Database Changes
API Changes
If this PR modifies APIs:
Performance Impact
Documentation
Deployment Considerations
Checklist
Additional Notes
Any additional information that reviewers should know about this PR.