Closed JQvickWester closed 4 months ago
Thank you for your submission! We have a total of 19 PRs open right now, and we are working hard on all of them! We will take a look as soon as we can.
@JQvickWester Static Tests results for commit 11c22ce20737abe9ad7c17b60260e139626b295f - https://drive.google.com/file/d/1ugFW-JuJwG6pnLemNmSxQ1qaBVKS61ui/view?usp=drivesdk
@JQvickWester Compile Tests results for commit 11c22ce20737abe9ad7c17b60260e139626b295f - https://drive.google.com/file/d/16YTMtWdwd7fXuAFfObBbF4RKSNFq89yn/view?usp=drivesdk
@JQvickWester Static Tests results for commit 9bf89b76bb9809c0fce3cd933723f4b44c2fadfe - https://drive.google.com/file/d/1N7kcqaGjttyY-KDcscj-QtN0LAP4NDHy/view?usp=drivesdk
@JQvickWester Compile Tests results for commit 9bf89b76bb9809c0fce3cd933723f4b44c2fadfe - https://drive.google.com/file/d/1Hzd-BVhq2uDFocfYltxS72q0p1ac93N6/view?usp=drivesdk
@JQvickWester Static Tests results for commit 5a13156992daa07b54df0d39d643ebbc0f1b1595 - https://drive.google.com/file/d/1QPbo999qjYV6TnfFAYPa1PbKY-KgPBe9/view?usp=drivesdk
@JQvickWester Compile Tests results for commit 5a13156992daa07b54df0d39d643ebbc0f1b1595 - https://drive.google.com/file/d/1P1_03ZBAIzJOioSojk511JXXeyThe3-p/view?usp=drivesdk
@JQvickWester Static Tests results for commit a8b7b33bd570189bc909331a17cd242b1fbf8a83 - https://drive.google.com/file/d/1cJSO0IzNI9bMj8u_EQKWYQiRcRXlZvlt/view?usp=drivesdk
@JQvickWester Static Tests results for commit d875613e8696ae622c873ae690fe8487cf8570c2 - https://drive.google.com/file/d/1QAEbKtG2-6Fo_dMubo7nQqJxXhSW3aMJ/view?usp=drivesdk
@JQvickWester Compile Tests results for commit d875613e8696ae622c873ae690fe8487cf8570c2 - https://drive.google.com/file/d/12H2xnlT8LF-jc29G-M9FOkbO5uvK6RsJ/view?usp=drivesdk
@JQvickWester Static Tests results for commit a9c66503a433d4f85131c7b526fab47e0290ec7b - https://drive.google.com/file/d/1TnaG4X2iwaviROw1tu-VjBdI0J9LdXX6/view?usp=drivesdk
@JQvickWester Compile Tests results for commit a9c66503a433d4f85131c7b526fab47e0290ec7b - https://drive.google.com/file/d/1f-sehQU-frwleQgnBAtfuhZvLetlf-Am/view?usp=drivesdk
@JQvickWester Static Tests results for commit a13adb6b6ff284719548456c26775fb3c41e3b1b - https://drive.google.com/file/d/1Ln2NSasEaTsXigg6bIYJ-O-oa3ptJNMH/view?usp=drivesdk
@JQvickWester Compile Tests results for commit a13adb6b6ff284719548456c26775fb3c41e3b1b - https://drive.google.com/file/d/1PPN4BFOyDyxjH6l-CEuxe06FdBaf6jPR/view?usp=drivesdk
@JQvickWester Static Tests results for commit 311bad10d6f3216b1060c67faeabb7e43bd98028 - https://drive.google.com/file/d/185iNrmFbibMwRXMafqDpyKajZ-XWXvmy/view?usp=drivesdk
@JQvickWester Compile Tests results for commit 311bad10d6f3216b1060c67faeabb7e43bd98028 - https://drive.google.com/file/d/1kwQ4lSWbqFRSKQmmZLWt9RPEpfSeY0Bv/view?usp=drivesdk
@JQvickWester Static Tests results for commit ea078b1eadbefcd42b3f4bafbcb8d2e38e82e6a0 - https://drive.google.com/file/d/1-Uxu0_lCEb_9pGFgXpuaM196DUFxn80h/view?usp=drivesdk
@JQvickWester Compile Tests results for commit ea078b1eadbefcd42b3f4bafbcb8d2e38e82e6a0 - https://drive.google.com/file/d/1SznFe9oBuqHdvpAAHF52X3fCwJ802xMw/view?usp=drivesdk
@JQvickWester Static Tests results for commit 2897bbf9a651184e384b7b8a19537e2bc77ee3c9 - https://drive.google.com/file/d/1eUR4yP6GcbX9p0BUjACz4GTkXU4OtAvQ/view?usp=drivesdk
@JQvickWester Compile Tests results for commit 2897bbf9a651184e384b7b8a19537e2bc77ee3c9 - https://drive.google.com/file/d/1pXINMDxzzN_Mr6LVb_5XxjEiOan7aSj1/view?usp=drivesdk
@mishalp-crest Static Tests results for commit 2a18d94fb30fb817a645f652c37ebab9b9144bff - https://drive.google.com/file/d/1jhc-QuNWaOY1Abpr9D8nuIcyDbLt0d8z/view?usp=drivesdk
@mishalp-crest Compile Tests results for commit 2a18d94fb30fb817a645f652c37ebab9b9144bff - https://drive.google.com/file/d/1Z33NETvzoAD8QKLm8v04wMq22INLqobo/view?usp=drivesdk
@JQvickWester
We usually update the app's minor version when there is a bug fix. Let us know if you want to update it.
@mishalp-crest Static Tests results for commit c79e4feee6b81c4765505afc48066ab60b024393 - https://drive.google.com/file/d/1zUsYjQXnRn9idb2W5Jz9hX06QBxVQ5MC/view?usp=drivesdk
@mishalp-crest Compile Tests results for commit c79e4feee6b81c4765505afc48066ab60b024393 - https://drive.google.com/file/d/1FyA9k46531sSBYxSQFEHwcX_kEtNHj-_/view?usp=drivesdk
@JQvickWester Can you please confirm?
Hello @mishalp-crest, I'm colleague of @JQvickWester, he's currently on long vacation. Why did you change the publisher from Recorded Future, Inc
to Splunk
in this commit? And also why did you decrease the min_phantom_version
in the same commit?
@mishalp-crest Static Tests results for commit eb812c58a54103a3d5ca9a217af028140a80e179 - https://drive.google.com/file/d/1bhwfAqUWUCRUSSdxM9xGHPgC91zP-EcW/view?usp=drivesdk
@mishalp-crest Static Tests results for commit a4dc68e121dc35b0aa8c18e5b2bb7d1449539f92 - https://drive.google.com/file/d/1KwKNOyGxG2u2M_J3LaYd14pKLv0HUYFE/view?usp=drivesdk
Hello @mishalp-crest, I'm colleague of @JQvickWester, he's currently on long vacation. Why did you change the publisher from
Recorded Future, Inc
toSplunk
in this commit? And also why did you decrease themin_phantom_version
in the same commit?
Reverted those changes
@mishalp-crest Compile Tests results for commit a4dc68e121dc35b0aa8c18e5b2bb7d1449539f92 - https://drive.google.com/file/d/10TOjppRYbysBTrpDRUdm1Ses6LJekTYK/view?usp=drivesdk
@mishalp-crest could you please apply my suggestions for version and also revert the changes to min_phantom_version
again? It should be 6.2.1
and not 6.1.1
. Thank you!
@mishalp-crest Static Tests results for commit 4a5d19fe910b5efbebaa10d5e4785f5941327135 - https://drive.google.com/file/d/1mn6TTpvS41tdAbrvs2PCDY9QIIWcy16u/view?usp=drivesdk
@mishalp-crest Compile Tests results for commit 4a5d19fe910b5efbebaa10d5e4785f5941327135 - https://drive.google.com/file/d/195CfJu9KKbfTngVJ6hfY0bUTQqusxKPQ/view?usp=drivesdk
@nplebanskyi Done with the changes let us know if you have any concern / doubt
@mishalp-crest all looks good, thank you.
Please ensure your pull request (PR) adheres to the following guidelines:
Pull Request Checklist
Please check if your PR fulfills the following requirements:
<App Name>: <PR Type> - <PR Description>
next
branch of the forked repo. Create separate feature branch for raising the PR.Pull Request Type
Please check the type of change your PR introduces:
Security Considerations (REQUIRED)
manual_readme_content.md
.manual_readme_content.md
all methods (eg, OAuth) used to authenticate with the service that the connector is integrating with.manual_readme_content.md
.nosemgrep
? If yes, please provide justification in an additional comment next to the ignored code.Release Notes (REQUIRED)
Unreleased
What is the current behavior? (OPTIONAL)
What is the new behavior? (OPTIONAL)
Other information (OPTIONAL)
Pay close attention to (OPTIONAL)
Screenshots (if relevant)
Thanks for contributing!