-
Notifications
You must be signed in to change notification settings - Fork 2
Feature: Allowing for batch file creation when the generated KPP file exceeds the file size limit #21
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
Open
astride
wants to merge
26
commits into
main
Choose a base branch
from
feature/didop-7690-splitlargefiles
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Feature: Allowing for batch file creation when the generated KPP file exceeds the file size limit #21
Changes from all commits
Commits
Show all changes
26 commits
Select commit
Hold shift + click to select a range
da0ea8b
Moving message-related helper methods to a MessageUtils class
astride 6d027d9
Customizing MessageUtils to allow for the BatchMessageUtils class to …
astride 106c699
Generating an XML document prior to saving the file to disk, to allow…
astride b45d5ee
Changing the order of events in KppService.Run() and introducing a he…
astride e236964
Introducing config values in the GUI project to allow for enabling/di…
astride 1050929
Implementing logic to create batch messages when necessary and possible
astride c40e837
Removing unnecessary XML helper method
astride 43a2034
Creating helper method for building test input file path
astride 20233fc
Renaming episode.csv to episode_institusjoner.csv and adding an episo…
astride adb7881
Adding batch file tests
astride 48e1783
Creating a TestBase class
astride e1cb03f
Moving the batch file tests to a separate class
astride 973b4d9
Using a file-scoped namespace for the test classes
astride 725af44
Moving the 'recommended batch file count' calculation to BatchMessage…
astride 483ff1a
Using more appropriate (i.e. larger) test files
astride 92689dc
Leaving lopenr out of the batch file name
astride d8f3a12
Setting gigabytes rather than bytes in the config file, and mocking t…
astride 3d57f78
Moving the batch file configuration from settings to the GUI form
astride 5777032
Implementing the possibility of creating batch files in the console app
astride be8cf03
Updating docs and readme files
astride 2c8f1f0
Making small fixes and removing TODOS
astride 28d36e2
Updating the assembly version / EPJ version to 1.9.0
astride 1aec8a3
Moving the BaseFileSizeInBytes constant from BatchMessageUtils to con…
astride e00102e
Reverting the 'serialize content to an XML document before saving to …
astride 98f54ad
Deleting the single KPP file if batch files should be created
astride 5f7853f
Bugfixes
astride File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
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
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
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
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,7 @@ | ||
| <?xml version="1.0" encoding="utf-8" ?> | ||
| <configuration> | ||
| <appSettings> | ||
| <add key="BaseFileSizeInBytes" value="3000" /> | ||
| <add key="BaseFileSizeInBytes_Info" value="The base file size is slightly larger than the base size of the generated XML file (before the file is populated with episode content)" /> | ||
| </appSettings> | ||
| </configuration> |
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
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
Oops, something went wrong.
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.
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.
De to
batchfiles-*-input-argumentene er bredere enn de eksisterende argumentene. Når man kjørerfor å vise veiledning for konsollapplikasjonen, ser det ikke så pent ut:
Har forsøkt å se på om det går an å konfigurere
Mono.Options.OptionSetellerMono.Options.OptionSet.WriteOptionDescriptions()for å øke avstanden mellom første og andre kolonne, men har ikke blitt klok på om det er mulig.