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
[iOS] Allow to send email with logs on iOS #8124
Conversation
Pull Request Checklist
|
Jenkins BuildsClick to see older builds (46)
|
018bd8b
to
7bd83af
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
a few questions
|
||
NSURL *dbFile = [logsFolderName URLByAppendingPathComponent:@"db.json"]; | ||
#if DEBUG | ||
NSString *networkDirPath = @"ethereum/mainnet_rpc_dev"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hmm, why do we limit to mainnet only?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, till we fix that encoding in status-go api method. That’s rather temporary thing to make sure we get at least some data in email on iOS
#endif | ||
|
||
NSURL *networkDir = [rootUrl URLByAppendingPathComponent:networkDirPath]; | ||
NSURL *originalGethLogsFile = [networkDir URLByAppendingPathComponent:@"geth.log"]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why don't we use ExportNodeLogs
here? https://github.com/status-im/status-go/blob/develop/mobile/status.go#L640
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Because of encoding, I wrote on Friday, forgot to remind about this on call today
|
98% of end-end tests have passed
Failed tests (1)Click to expand
Passed tests (48)Click to expand |
100% of end-end tests have passed
Passed tests (1) |
@rasom |
@churik are you sure you installed build №10? I can't reproduce it. |
@rasom some glitch in #8124 (comment) (last builds were #9)
Checked that doesn't affect to Android, so I can send logs. |
fetching status-go logs using
StatusgoExportNodeLogs()
will be added in the separate PR, requires status-go update so might take some extra timestatus: ready