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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Modified the results report #560

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Mrinank-Bhowmick
Copy link
Contributor

Motivation and Context

For an impressive user experience 馃

Modified the results report template by adding OWASP logo at the bottom to make it look more professional.

  • Screenshot :
    Screenshot-2022-04-14-222846.jpg

Reviewers

@Ali-Razmjoo
@securestep9

Checklist

  • I have followed the Contributor Guidelines.
  • The code has been thoroughly tested in my local development environment with flake8 and pylint.
  • The code is Python 3 compatible.
  • The code follows the PEP8 styling guidelines with 4 spaces indentation.
  • This Pull Request relates to only one issue or only one feature
  • I have referenced the corresponding issue number in my commit message
  • I have added the relevant documentation.
  • My branch is up-to-date with the Upstream master branch.

Changes proposed in this pull request

Your development environment

  • OS: Docker: Debian GNU/Linux 11 (bullseye)
  • OS Version: 11
  • Python Version: 3.9

@securestep9
Copy link
Collaborator

sorry @Mrinank-Bhowmick we have a few more important functional enhancements coming to the report, your proposal is purely cosmetic. I am going to keep this PR open as a remind to ask you to check again once the report has been modified

@Mrinank-Bhowmick
Copy link
Contributor Author

sorry @Mrinank-Bhowmick we have a few more important functional enhancements coming to the report, your proposal is purely cosmetic. I am going to keep this PR open as a remind to ask you to check again once the report has been modified

Sure , I will check again and update it.

@Mrinank-Bhowmick Mrinank-Bhowmick marked this pull request as draft October 27, 2022 08:58
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.

None yet

2 participants