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

Add additional logging #188

Merged
merged 5 commits into from
Jan 26, 2021
Merged

Add additional logging #188

merged 5 commits into from
Jan 26, 2021

Conversation

MakMuftic
Copy link
Member

@MakMuftic MakMuftic commented Jan 26, 2021

Add additional logging

@mpetrunic I think there is no need to add this to the changelog file?
We should actually as we are going to release new version

PR Checklist

  • I have run linter localy
  • I have run unit and integration tests locally
  • Rebased to master branch / merged master
  • Updated CHANGELOG.md

Changes

  • Add reason why the node is on cooldown
  • Fix wrong error display on connection error (ws)
  • Update docker-compose file

Issues

Closes #187

@MakMuftic
Copy link
Member Author

@mpetrun5 @mpetrunic I also bumped the version to v0.5.2 here

mpetrunic
mpetrunic previously approved these changes Jan 26, 2021
docker-compose.yml Outdated Show resolved Hide resolved
Co-authored-by: Marin Petrunić <mpetrunic@users.noreply.github.com>
@mpetrunic mpetrunic merged commit 52da871 into dev Jan 26, 2021
@mpetrunic mpetrunic deleted the mmuftic/additional-logging branch January 26, 2021 12:26
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.

Display reason in logs why node is on cooldown
3 participants