-
Notifications
You must be signed in to change notification settings - Fork 228
Updated the Airtable app doc #5428
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
Conversation
|
Approved, @amee-sumo. But I'm not sure why the package.json and yarn.lock files are updated by this PR, when they should have only been updated by PR #5420. Maybe @kimsauce can tell us. |
That's strange. While it doesn't hurt anything because there’s no 3.8.1 or newer yet, it could cause unpredictable changes later. @amee-sumo , I'd remove the ^ and pin versions exactly. You can...
|
I made a commit with the required changes @kimsauce @jpipkin1. Please acknowledge if we are good to merge this PR. |
|
@amee-sumo, I checked the PR, but still saw the package.json and yarn.lock files there. So I fixed the package.json file. But when I tried to remove and regenerate the yarn.lock file using Kim's command, the yarn.lock file still ended up appearing on the PR. At this point, I recommend closing this PR and making a fresh one with your changes to the Airtable app article. |
…gic-documentation into update-airtable-app-doc
@amee-sumo @jpipkin1 All good! I just reverted the yarn.lock to what it is now. |
|
Thanks, @kimsauce! |
Purpose of this pull request
This pull request is to update the Airtable app doc with new content in the log collection, dashboard, and monitors section.
Select the type of change
Ticket (if applicable)
https://sumologic.atlassian.net/browse/DOCS-904