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

Small typo in BloodHound customqueries.json: change comma to colon #398

Closed
wants to merge 1 commit into from

Conversation

itsnotgunnar
Copy link

There was a small typo in one of the queries that brought up a graph error. Changed the comma to a colon.

Thanks for putting out the configuration.

@gbe
Copy link
Contributor

gbe commented Oct 8, 2024

Thanks for your PR.
Unless I'm wrong, this issue was fixed in #362

@itsnotgunnar
Copy link
Author

Hey that's my bad. I was referring to your main branch's file for my configuration, then changed the pull request target to the dev branch after I read the contribution.md. Still learning how this works.. will refer to past pull requests first. Thanks again.

@ShutdownRepo
Copy link
Member

Thank you @itsnotgunnar for this PR, even if it turns out to be a duplicate 😅
Thank you @gbe for identifying the dup!

@ShutdownRepo ShutdownRepo added the duplicate This issue or pull request already exists label Oct 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate This issue or pull request already exists
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants