Skip to content

new code block with error handling LOG-9481#29

Open
tanberry wants to merge 7 commits into
mainfrom
log-9481
Open

new code block with error handling LOG-9481#29
tanberry wants to merge 7 commits into
mainfrom
log-9481

Conversation

@tanberry

Copy link
Copy Markdown

Per Jira LOG-9481, Braxton reworked code example to include error handling best practice.

@tanberry tanberry requested review from LYHuang and darinspivey April 16, 2021 02:15
Comment thread README.md Outdated
trace_id: '163e169e-a326-4b1b-a8f5-7169dd4eeca8',
}
logger.log({
message: JSON.stringify(log_obj_info), // Optional. If not provided, the stringified (read JSON) object in it's entirety will be sent as the payload

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
message: JSON.stringify(log_obj_info), // Optional. If not provided, the stringified (read JSON) object in it's entirety will be sent as the payload
message: JSON.stringify(log_obj_info), // Optional. If not provided, the stringified (read JSON) object in its entirety will be sent as the payload

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Gracious thank you for catching that. I literally just swapped out the code block and didn't look at rest of doc.

Comment thread README.md Outdated
Tana M Berry and others added 2 commits April 16, 2021 08:53
Co-authored-by: Darin Spivey <darinspivey@users.noreply.github.com>
Co-authored-by: Darin Spivey <darinspivey@users.noreply.github.com>
@darinspivey

Copy link
Copy Markdown
Contributor

What happened to the other change? There's only 1 now. Did you force push over it, @tanberry ?

tanberry added 2 commits April 16, 2021 10:55
Merge branch 'log-9481' of github.com:logdna/logdna-winston into log-9481
@tanberry

Copy link
Copy Markdown
Author

Sorry about that @darinspivey; I attempted to use the "Commit suggestion" button on the UI for both of your edits, and I don't think it worked for one of them. SoI made the changes locally and pushed again. No for sure I don't force push anything, not that confident in here yet. :-)

@tanberry tanberry requested a review from darinspivey April 16, 2021 16:13
darinspivey
darinspivey previously approved these changes Apr 16, 2021
@tanberry

Copy link
Copy Markdown
Author

Ah, I just saw in the Jira the new/updated code chunk. OK, will add that in next.

@braxtonj

Copy link
Copy Markdown

hey @darinspivey, i changed it last minute on Tana and caused some confusion 🤦. What's in the Jira now has the correct modifications. Changed ordering, comments, a little structure and added a warn example

darinspivey
darinspivey previously approved these changes Apr 16, 2021
@tanberry

Copy link
Copy Markdown
Author

okeeeee now this PR has the latest code block, the whole thing, in it, and is ready for review/approval/merging. ;-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants