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

Adding in the Controls and DatastoreType classes to documentation #201

Merged

Conversation

jharnois4512
Copy link
Contributor

@jharnois4512 jharnois4512 commented Jul 28, 2022

Figured out how to make this PR with Raphael's response to my issue I opened. I noticed that the documentation was missing the Controls class. When I went to make the changes I also noticed it was missing the DatastoreType class as well. So both of these are in the PR.

Any comments, let me know!

@jharnois4512 jharnois4512 requested a review from izar as a code owner July 28, 2022 11:39
@ghost
Copy link

ghost commented Jul 28, 2022

👇 Click on the image for a new way to code review
  • Make big changes easier — review code in small groups of related files

  • Know where to start — see the whole change at a glance

  • Take a code tour — explore the change with an interactive tour

  • Make comments and review — all fully sync’ed with github

    Try it now!

Review these changes using an interactive CodeSee Map

Legend

CodeSee Map Legend

@izar izar merged commit 7327432 into OWASP:master Jul 28, 2022
@izar
Copy link
Collaborator

izar commented Jul 28, 2022

Thanks! Much appreciated!

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.

2 participants