Skip to content

local html reports improved #592

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.

Sign up for GitHub

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

Jump to bottom
Merged
securestep9 merged 8 commits into from
Sep 24, 2022
Merged

local html reports improved #592

securestep9 merged 8 commits into from
Sep 24, 2022

Conversation

@itsdivyanshjain
Copy link
Collaborator

@itsdivyanshjain itsdivyanshjain commented Sep 18, 2022

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

Improved html local reports

Your development environment

  • OS: x
  • OS Version: x
  • Python Version: x

@Ali-Razmjoo
Copy link
Collaborator

Ali-Razmjoo commented Sep 24, 2022

The only issue I saw was when I tried to copy the JSON content; the Unicode came with it. Maybe there should be a copy to the clipboard button.

▼{
    "timeout": 3,
    "headers": ▼{
        "User-Agent": "Nettacker 0.0.3 TRES"
    },
    "allow_redirects": false,
    "url": "http://newsroom.uber.com:80/wp-content/plugins/akismet/readme.txt",
    "method": "get",
    "response": ▼{
        "conditions_results": ▼{
            "content": ▼[
                "Tags:"
            ],
            "status_code": ▼[
                "200"
            ]
        }
    }
}

@securestep9 securestep9 self-assigned this Sep 24, 2022
@securestep9 securestep9 self-requested a review September 24, 2022 17:45
@securestep9 securestep9 merged commit 02d56f1 into master Sep 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@securestep9 securestep9 securestep9 approved these changes

Assignees

@securestep9 securestep9

Labels

enhancement

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@itsdivyanshjain @Ali-Razmjoo @securestep9