-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Copy receipt changes to 1.3 #199
Conversation
- Use SQLiteJournalFRCopyPayloadRepository in CopyCommand - Simplify method GetCountOfExistingCopiesAsync for efficiency
…FRCopyPayloadRepository interface and fixed related issues
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Still missing is the PopulateFtJournalFRCopyPayloadTable
functionality.
I'd like that to happen when the Migration happens but we haven't done something similar before so theres no precendent how to do it 😅
Also when its done fot sqlite we'll need to add that functionality to the other storages aswell (I'd do that in the end).
queue/src/fiskaltrust.Middleware.Contracts/Repositories/FR/IJournalFRCopyPayloadRepository.cs
Outdated
Show resolved
Hide resolved
queue/src/fiskaltrust.Middleware.Localization.QueueFR/RequestCommands/CopyCommand.cs
Show resolved
Hide resolved
queue/src/fiskaltrust.Middleware.Localization.QueueFR/RequestCommands/CopyCommand.cs
Outdated
Show resolved
Hide resolved
queue/src/fiskaltrust.Middleware.Localization.QueueFR/RequestCommands/CopyCommand.cs
Outdated
Show resolved
Hide resolved
queue/src/fiskaltrust.Middleware.Storage.SQLite/Migrations/012_ftJournalFRCopyPayload.sqlite3
Outdated
Show resolved
Hide resolved
...iskaltrust.Middleware.Storage.SQLite/Repositories/FR/SQLiteJournalFRCopyPayloadRepository.cs
Outdated
Show resolved
Hide resolved
...iskaltrust.Middleware.Storage.SQLite/Repositories/FR/SQLiteJournalFRCopyPayloadRepository.cs
Outdated
Show resolved
Hide resolved
- Modified PopulateFtJournalFRCopyPayloadTableAsync for correct deserialization and conversion of payloads - Added a helper class for conversion between ftJournalFRCopyPayload and ftJournalFR - Optimization and cleanup
…: JournalConverter
…erify the existence of the ftJournalFRCopyPayload table after all migrations
queue/src/fiskaltrust.Middleware.Localization.QueueFR/RequestCommands/CopyCommand.cs
Outdated
Show resolved
Hide resolved
queue/src/fiskaltrust.Middleware.Storage.Base/BaseStorageBootStrapper.cs
Outdated
Show resolved
Hide resolved
queue/src/fiskaltrust.Middleware.Storage.SQLite/DatabaseInitialization/DatabaseMigrator.cs
Outdated
Show resolved
Hide resolved
queue/src/fiskaltrust.Middleware.Storage.SQLite/SQLiteStorageBootstrapper.cs
Outdated
Show resolved
Hide resolved
…sitory interface correctly
… EFCore for PostgreSQL and SQLServer.
Use query parameters in mysql migrations
queue/test/fiskaltrust.Middleware.Localization.QueueFR.UnitTest/BaseStorageBootStrapperTests.cs
Show resolved
Hide resolved
…ust/middleware into add-copy-receipt-changes-to-1.3
…QL and EFCoreSQLServer
…EFCore conventions
{Summary of the changes}
Description
{Detail}
Fixes #{issue number}