Merged
Conversation
Rework charts on Report Page into separate Tab Views Smaller refactorings to initialize BucketListingViewModel
The issue in #317 has reappeared for me, especially on mobile (Firefox Android). Fix: MudBlazor/MudBlazor#8746 (comment) For me, only the InfoDialog is affected (maybe because there is nothing to focus on?), we could also disable this globally by setting `MudGlobal.DialogDefaults.DefaultFocus = DefaultFocus.None`
Removed Microsoft.VisualStudio.Azure.Containers.Tools.Targets
Fix table header title in RecurringTransactionDialog
TheAxelander
changed the title
Contributor
There was a problem hiding this comment.
Pull request overview
This pull request prepares version 1.11 for release with several significant enhancements and refactorings:
- Added authentication framework with configurable username/password support
- Upgraded Entity Framework Core from 8.0 to 9.0
- Refactored database initialization to use IDbContextFactory pattern
- Improved error handling with consistent logging throughout the application
- Modernized test infrastructure with collection expressions and in-memory database testing
- Added ReSharper/Rider code style configuration file
Reviewed changes
Copilot reviewed 152 out of 152 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| examples/example.env | Added authentication configuration parameters |
| SECURITY.md | Updated supported version to 1.11 |
| OpenBudgeteer.sln.DotSettings | Added ReSharper/Rider code style settings |
| OpenBudgeteer.sln | Referenced new DotSettings file |
| ViewModelBase.cs | Added ILogger dependency to base class |
| Multiple ViewModel files | Updated constructors to accept and use logger |
| Multiple PageViewModel files | Added ServiceException handling and logging |
| Database initialization files | Refactored to use IDatabaseConnector interface |
| EFCore service files | Migrated to IDbContextFactory pattern with logging |
| Generic service files | Improved error messages and exception handling |
| Test files | Converted to collection expressions, switched to in-memory database |
| Package references | Upgraded EF Core and related packages to version 9.0 |
| CSS files | Added styling for MudBlazor components |
| RedirectToLogin.razor | New authentication redirect component |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.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
2 participants
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.
Check list:
TODOin code