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.

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

SUMO-227719: added documentation for first set of webhook apps #2976

Merged
merged 10 commits into from
Aug 24, 2023
Merged

SUMO-227719: added documentation for first set of webhook apps #2976

merged 10 commits into from
Aug 24, 2023

Conversation

hitarth-sumo
Copy link
Contributor

@hitarth-sumo hitarth-sumo commented Aug 23, 2023 •
edited by kimsauce
Loading

Purpose of this pull request

This pull request (PR) ...

Issue number:

Select the type of change:

  • Minor Changes - Typos, formatting, slight revisions
  • Update Content - Revisions and updating sections
  • New Content - New features, sections, pages, tutorials
  • Site and Tools - Updates, maintenance, and new packages for the site, Gatsby, React, etc

@cla-bot cla-bot bot added the cla-signed Contributor approved, listed in .clabot file label Aug 23, 2023
@jpipkin1 jpipkin1 added the doc:new New section or topic to content label Aug 23, 2023
jpipkin1
jpipkin1 approved these changes Aug 23, 2023
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, @hitarth-sumo. I found only a few tiny things that I went ahead and fixed. These are very well-written articles. Thank you for taking the time to make sure they were correct.

@jpipkin1
Copy link
Collaborator

@kimsauce, should we go ahead and publish this PR?

@jpipkin1 jpipkin1 enabled auto-merge August 24, 2023 13:59
@jpipkin1 jpipkin1 added this pull request to the merge queue Aug 24, 2023
Merged via the queue into SumoLogic:main with commit 3783d9e Aug 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kimsauce kimsauce kimsauce left review comments

@jpipkin1 jpipkin1 jpipkin1 approved these changes

@stacykor stacykor Awaiting requested review from stacykor

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

Assignees

@hitarth-sumo hitarth-sumo

Labels
cla-signed Contributor approved, listed in .clabot file doc:new New section or topic to content
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@hitarth-sumo @jpipkin1 @kimsauce