Skip to content
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

Add adoc report generator #87

Merged
merged 16 commits into from
Sep 9, 2024

Conversation

CMon
Copy link
Contributor

@CMon CMon commented Jul 3, 2024

An Adoc report generator as alternative/addon to the current pdf generator.

This enables us to have some more theming possibilities or create reports with custom chapters. It also has the ability to automatically create tocs that are able to show up inside modern pdf viewers like requested in #84
adoc could also be used to create a html version of the report (requested in #49 )

@CMon CMon force-pushed the add-adoc-report-generator branch from be29b31 to 841cf35 Compare July 5, 2024 11:21
@CMon
Copy link
Contributor Author

CMon commented Jul 14, 2024

I currently have some open todos:

  • get all basic html code fixed/adjusted that is injected via modules (current open one is )
  • fix all small grey text "footnotes", some i missed during the first run

@CMon CMon force-pushed the add-adoc-report-generator branch from acf2792 to 093fe55 Compare July 26, 2024 12:33
@srcnuzn
Copy link

srcnuzn commented Sep 6, 2024

Hey @CMon are there any plans on when to merge your PR? I'm really looking forward to it.

@CMon
Copy link
Contributor Author

CMon commented Sep 9, 2024

Hi @srcnuzn, I have no permission to merge. I used this source to create a Threat Model and handed it to the TÜV for some certification lets hope they are happy with it, otherwise I have to ditch the tool anyway. Until someone with merge permissions merges this feature, you could still go with my forked version: fix-report-risk-status-error which beside of the number fix that is reported here: #83 also includes the adoc generator.

@srcnuzn
Copy link

srcnuzn commented Sep 9, 2024

I see. I will try out the forked version if I have some time. If you like, let us know how the certification process went (and which cert you went for). Maybe there is useful feedback for the development of this tool or for other users who evaluate it (like me 🙃).

@ezavgorodniy ezavgorodniy merged commit 4709f9e into Threagile:master Sep 9, 2024
6 checks passed
@ezavgorodniy
Copy link
Collaborator

Sorry for not coming back for a while, LGTM, just mergeed

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.

3 participants