Skip to content

#53 click on Save button save an archive with FilesToAttach instead of .txt - #54

Merged
PandaWood merged 2 commits into
PandaWood:masterfrom
makeProjectGreatAgain:zipReportService
Jun 13, 2021
Merged

#53 click on Save button save an archive with FilesToAttach instead of .txt#54
PandaWood merged 2 commits into
PandaWood:masterfrom
makeProjectGreatAgain:zipReportService

Conversation

@makeProjectGreatAgain

Copy link
Copy Markdown
Contributor

#53 click on Save button save the zip file, which contains FilesToAttach, screenshot and report.txt
Save Button

  • added ZipReportService.cs which makes zip file with FilesToAttach, screenshot and optional additionalFilesToAttach (for example, report.txt)
  • added Unit Tests for ZipReportService.cs

ExceptionReportView - SaveClick now save to zip file.
Attacher - rename File property to FileService.
@PandaWood

Copy link
Copy Markdown
Owner

This looks good. It makes sense that "Save" should save the zip archive and not a text file. Because that's the same format as what would have been sent via any other method of sending the report. This should really have been updated when the "Templating" feature was added - it just wasn't noticed.
I'll do a more careful review of the code and test it out shortly.

@PandaWood
PandaWood merged commit d8b820e into PandaWood:master Jun 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Proposal: click on Save button save an archive with FilesToAttach instead of .txt

2 participants