Cleaning this tp reduce code duplication and simplify logic
Details
Details
Play with app and make sure ops are processed and results are persisted.
Diff Detail
Diff Detail
- Repository
- rCOMM Comm
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
| web/database/sqlite-api.js | ||
|---|---|---|
| 199 ↗ | (On Diff #43450) | We were checking the drafts differently from the other ops. Are you sure the new logic is equivalent? |
| web/database/sqlite-api.js | ||
|---|---|---|
| 199 ↗ | (On Diff #43450) | Yes, this line is different because drafts are not converted and can be nullable. In the new code, this condition is inverted, we check if there is any field that exists and has a length higher than 0 so this should remain unchanged. |