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.

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

Update log format #44

Merged
merged 4 commits into from
Aug 19, 2024
Merged

Update log format #44

merged 4 commits into from
Aug 19, 2024

Conversation

shuse2
Copy link
Member

@shuse2 shuse2 commented Aug 19, 2024

What was the problem?

This PR resolves LISK-963

How was it solved?

  • Remove 2nd/3rd params from JSON.stringify
  • Update error to be part of the log message

How was it tested?

  • Run locally

@shuse2 shuse2 self-assigned this Aug 19, 2024
src/utils/LogUtils.ts Outdated Show resolved Hide resolved
@sameersubudhi sameersubudhi merged commit d9c1efd into master Aug 19, 2024
1 check passed
@sameersubudhi sameersubudhi deleted the LISK-963-update-log-format branch August 19, 2024 12:06
sameersubudhi pushed a commit that referenced this pull request Sep 6, 2024
### What was the problem?

This PR resolves LISK-963

### How was it solved?

* Remove 2nd/3rd params from JSON.stringify
* Update error to be part of the log message

### How was it tested?

* Run locally
sameersubudhi pushed a commit that referenced this pull request Sep 10, 2024
### What was the problem?

This PR resolves LISK-963

### How was it solved?

* Remove 2nd/3rd params from JSON.stringify
* Update error to be part of the log message

### How was it tested?

* Run locally
sameersubudhi pushed a commit that referenced this pull request Sep 10, 2024
### What was the problem?

This PR resolves LISK-963

### How was it solved?

* Remove 2nd/3rd params from JSON.stringify
* Update error to be part of the log message

### How was it tested?

* Run locally
sameersubudhi pushed a commit that referenced this pull request Sep 10, 2024
### What was the problem?

This PR resolves LISK-963

### How was it solved?

* Remove 2nd/3rd params from JSON.stringify
* Update error to be part of the log message

### How was it tested?

* Run locally
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