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

Improve report rendering #81

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

psyray
Copy link
Contributor

@psyray psyray commented Mar 5, 2023

  • Refactor CreateReport Controller to prevent code duplication and improve maintenability
  • Add 2 methods to generate PHPWord (markdownParser & setValue)
  • Add null and empty count checks to prevent errors on generation if datas is missing
  • Add new line in HTML when there are some in Markdown

Tested on the latest version

psyray added 4 commits March 5, 2023 23:36
- Refactor CreateReport Controller to prevent code duplication and
improve maintenability
- Add 2 methods to generate PHPWord (markdownParser & setValue)
- Add null and empty count checks to prevent errors on generation if
datas is missing
- Add new line in HTML when there are some in Markdown
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.

1 participant