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

Reporting templates, cleanup #213

Merged
merged 15 commits into from Jan 26, 2022
Merged

Reporting templates, cleanup #213

merged 15 commits into from Jan 26, 2022

Conversation

m-1-k-3
Copy link
Member

@m-1-k-3 m-1-k-3 commented Jan 26, 2022

  • What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

Feature

  • What is the current behavior? (You can also link to an open issue here)

No further details on the tasks of the modules included.
No user defined templates for reports available
Missing PR template
Copyright info is not up to date
sometimes emba, sometimes EMBA
No random redis password set during installation
Missing CODE_OF_CONDUCT.md

  • What is the new behavior (if this is a feature change)? If possible add a screenshot.

Introduction of reporting templates - new template which includes basic description in template files. pre templates are used at the beginning of a module, post templates are used at the end of the module. Templates can be txt or sh files.
Random redis password set during installation
Added CODE_OF_CONDUCT.md

  • Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)

Hopefully not.

  • Other information:
    Screenshot of template usage:

emba_post-reporting

@m-1-k-3 m-1-k-3 added the enhancement New feature or request label Jan 26, 2022
@m-1-k-3 m-1-k-3 requested a review from p4cx January 26, 2022 13:26
@p4cx p4cx merged commit fdad9fb into e-m-b-a:master Jan 26, 2022
@m-1-k-3 m-1-k-3 mentioned this pull request Jan 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants