Depends on D3585
Add logic for storing the log data in the database.
Differential D3586
[services] Backup - Send log - add store in database • karol on Mar 31 2022, 6:32 AM. Authored by Tags None Referenced Files
Details
Diff Detail
Event Timeline
Comment Actions Please make the summary more descriptive. Add the rest of SendLog functionality is a really general description of what is happening here. And a lot is happening here - is there a way to split this diff? (It's also confusing what do you mean by the rest - it doesn't sound like adding new todos should be a part of a diff which adds the rest).
A diff like that should definitely have a concrete test plan - it looks like at this point the logic should be easy to test Comment Actions Ok, I split the diff into multiple diffs. I hope it's now easier for you to review, I was trying to add a simple piece of code in every diff. The test plan you'd probably want to see is in D3651 Comment Actions Thanks! It's a lot better!
|