Skip to content

Conversation

kiblik
Copy link
Contributor

@kiblik kiblik commented Nov 27, 2023

Small rewrite to more compact a readable ViewSets.

More readable code ~ less bugs

Note: There is no effect on the API schema.

@github-actions github-actions bot added the apiv2 label Nov 27, 2023
Copy link

dryrunsecurity bot commented Nov 27, 2023

Contextual Security Analysis

As DryRun Security performs checks, we’ll summarize them here. You can always dive into the results in the section below for checks.

Status DryRun Security Check
Configured Sensitive Files Check
AI-powered Sensitive Files Check

Chat with your AI-powered Security Buddy by typing /dryrunsec: (or /drs:) followed by your question. Example: /dryrunsec: From a security perspective, what are some sensitive files in an Express application?

Install and configure more repositories at DryRun Security

@kiblik kiblik changed the title API: Use better ViewSet API: Use more readable ViewSets Nov 28, 2023
@kiblik kiblik force-pushed the api_viewset_cleanup branch from f0e6711 to 0d7c4f9 Compare November 28, 2023 10:09
@kiblik kiblik mentioned this pull request Nov 28, 2023
5 tasks
@kiblik kiblik marked this pull request as ready for review November 28, 2023 10:44
Copy link
Contributor

@mtesauro mtesauro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved

@blakeaowens blakeaowens merged commit 8172ce2 into DefectDojo:dev Dec 12, 2023
@kiblik kiblik deleted the api_viewset_cleanup branch December 12, 2023 22:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants