Page MenuHomePhabricator

Reuse more transaction construction code in bulk editor
ClosedPublic

Authored by epriestley on Jan 19 2018, 1:53 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, May 3, 5:05 AM
Unknown Object (File)
Thu, May 2, 12:53 AM
Unknown Object (File)
Thu, Apr 25, 12:45 AM
Unknown Object (File)
Fri, Apr 19, 7:04 PM
Unknown Object (File)
Thu, Apr 18, 7:16 AM
Unknown Object (File)
Thu, Apr 11, 8:35 AM
Unknown Object (File)
Sat, Apr 6, 12:19 AM
Unknown Object (File)
Apr 2 2024, 3:16 AM
Subscribers
None

Details

Summary

Ref T13025. Currently, the bulk editor takes an HTTP request and emits a list of "raw" transactions (simple dictionaries). This goes into the job queue, and the background job builds a real transaction.

However, the logic to turn an HTTP request into a raw transaction is ending up with some duplication, since we generally already have logic to turn an HTTP request into a full object.

Instead: build real objects first, then serialize them to dictionaries. Send those to the job queue, rebuild them into objects again, and we end up in the same spot with a little less code duplication.

Finally, delete the mostly-copied code.

Test Plan

Used bulk editor to add comments, projects, and rename tasks.

Diff Detail

Repository
rP Phabricator
Branch
bulk10
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 19051
Build 25707: Run Core Tests
Build 25706: arc lint + arc unit