Skip to content

V1 to v2 apps migration (Release_2) + Threat Intel #4157

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
merged 30 commits into from
May 2, 2025
Merged

V1 to v2 apps migration (Release_2) + Threat Intel #4157

merged 30 commits into from
May 2, 2025

Conversation

@JV0812
Copy link
Collaborator

@JV0812 JV0812 commented Jun 3, 2024 •
edited
Loading

Purpose of this pull request

This pull request is to update the app installation steps as per V2. Also, add updating and deleting the app information.

Select the type of change

  • Minor Changes - Typos, formatting, slight revisions
  • Update Content - Revisions, updating sections
  • New Content - New features, sections, pages, tutorials
  • Site and Tools - .clabot, version updates, maintenance, dependencies, new packages for the site (Docusaurus, Gatsby, React, etc.)

Ticket (if applicable)

Doc changes for the migrated V2 apps which are using Threat Intel

@cla-bot cla-bot bot added the cla-signed Contributor approved, listed in .clabot file label Jun 3, 2024
@JV0812 JV0812 self-assigned this Jun 3, 2024
@JV0812 JV0812 added the do-not-merge Pull requests that should not be merged label Jun 3, 2024
kimsauce
kimsauce reviewed Jun 3, 2024
View reviewed changes
kimsauce
kimsauce reviewed Jun 3, 2024
View reviewed changes
kimsauce
kimsauce reviewed Jun 3, 2024
View reviewed changes
kimsauce
kimsauce reviewed Jun 3, 2024
View reviewed changes
kimsauce
kimsauce reviewed Jun 3, 2024
View reviewed changes
kimsauce
kimsauce approved these changes Jun 3, 2024
View reviewed changes
Copy link
Collaborator

@kimsauce kimsauce left a comment

Choose a reason for hiding this comment

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

Great job! minor fixes

JV0812 and others added 9 commits June 5, 2024 10:22
@JV0812 @kimsauce
Update docs/integrations/security-threat-detection/threat-intel-quick…
7fac2b4 
…-analysis.md

Co-authored-by: Kim (Sumo Logic) <[email protected]>
@JV0812 @kimsauce
Update docs/integrations/security-threat-detection/threat-intel-quick…
7642d25 
…-analysis.md

Co-authored-by: Kim (Sumo Logic) <[email protected]>
@JV0812 @kimsauce
Update docs/integrations/security-threat-detection/threat-intel-quick…
ff61707 
…-analysis.md

Co-authored-by: Kim (Sumo Logic) <[email protected]>
@JV0812 @kimsauce
Update docs/integrations/security-threat-detection/threat-intel-quick…
85100cf 
…-analysis.md

Co-authored-by: Kim (Sumo Logic) <[email protected]>
@JV0812 @kimsauce
Update docs/integrations/security-threat-detection/threat-intel-quick…
a3dfbbc 
…-analysis.md

Co-authored-by: Kim (Sumo Logic) <[email protected]>
@JV0812 @kimsauce
Update docs/integrations/security-threat-detection/threat-intel-quick…
5d3e930 
…-analysis.md

Co-authored-by: Kim (Sumo Logic) <[email protected]>
@JV0812 @kimsauce
Update docs/integrations/security-threat-detection/threat-intel-quick…
fb9f26f 
…-analysis.md

Co-authored-by: Kim (Sumo Logic) <[email protected]>
@JV0812 @kimsauce
Update docs/integrations/saas-cloud/microsoft-exchange-trace-logs.md
c694176 
Co-authored-by: Kim (Sumo Logic) <[email protected]>
@JV0812 @kimsauce
Update docs/integrations/saas-cloud/gmail-tracelogs.md
10413b9 
Co-authored-by: Kim (Sumo Logic) <[email protected]>
JV0812
JV0812 commented Apr 4, 2025
View reviewed changes
@JV0812 JV0812 requested review from a team and amee-sumo as code owners April 4, 2025 05:50
@jpipkin1
Copy link
Collaborator

jpipkin1 commented Apr 4, 2025

@JV0812, this PR has some merge conflicts. Let me know when you've resolved them, and I'll give it a review.

jpipkin1
jpipkin1 approved these changes Apr 8, 2025
View reviewed changes
Copy link
Collaborator

@jpipkin1 jpipkin1 left a comment

Choose a reason for hiding this comment

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

Approved, @JV0812.

I removed the FAQ section from the Threat Intel article because it talks about CrowdStrike, and we are removing the CrowdStrike source from threat intel by the end of the month. See my commit "Remove FAQ section from Threat Intel Quick Analysis article".

kimsauce
kimsauce approved these changes Apr 8, 2025
View reviewed changes
Copy link
Collaborator

@kimsauce kimsauce left a comment

Choose a reason for hiding this comment

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

indentation - otherwise looks great

@soc-service-agent
Copy link

soc-service-agent commented Apr 25, 2025 •
edited
Loading

🎉 Snyk checks have passed. No issues have been found so far.

✅ security/snyk check is complete. No issues have been found. (View Details)

✅ license/snyk check is complete. No issues have been found. (View Details)

@JV0812 JV0812 removed the do-not-merge Pull requests that should not be merged label May 2, 2025
@JV0812 JV0812 added this pull request to the merge queue May 2, 2025
Merged via the queue into main with commit d304e71 May 2, 2025
7 checks passed
@JV0812 JV0812 deleted the V1-to-V2-apps-migration branch May 2, 2025 07:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kimsauce kimsauce kimsauce approved these changes

@jpipkin1 jpipkin1 jpipkin1 approved these changes

@mafsumo mafsumo Awaiting requested review from mafsumo mafsumo is a code owner

@amee-sumo amee-sumo Awaiting requested review from amee-sumo amee-sumo is a code owner

Assignees

@JV0812 JV0812

Labels

cla-signed Contributor approved, listed in .clabot file

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@JV0812 @jpipkin1 @soc-service-agent @kimsauce