Skip to content

Conversation

@majecty
Copy link
Contributor

@majecty majecty commented May 29, 2019

Example content

DB is connected
Agent: test
  address: Some(V4(127.0.0.1:3485))
  status: Run
  peer count: 3
  best block number: Some(100)
  available disk: 63802 MB
  memory usage: 1760 MB / 2040 MB
Agent: test2
  address: None
  status: Stop
  peer count: 0
  best block number: None
  available disk: 63802 MB
  memory usage: 0 MB / 0 MB

@majecty majecty added the server label May 29, 2019
@majecty majecty changed the title [WIP] Send daily reports Send daily reports May 30, 2019
@majecty majecty requested a review from sgkim126 May 30, 2019 11:52
@sgkim126
Copy link
Contributor

Looks good, but how about changing methods of Noti takes a slice of str?
It would help to format email notification.

@majecty
Copy link
Contributor Author

majecty commented May 31, 2019

Good! Then the "
' parts would be moved to the noti module. it would be a better capsulization.

@majecty
Copy link
Contributor Author

majecty commented May 31, 2019

I tested the email, just using the "\n" works in email. Maybe the default content type was not "html".
I'll use the original code.

@majecty majecty merged commit 201708e into CodeChain-io:master May 31, 2019
@majecty majecty deleted the f/daily-report branch May 31, 2019 05:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants