Skip to content
This repository has been archived by the owner on Mar 23, 2023. It is now read-only.

Rename TrackingBatch batch-specific fields to match Transact protocol Batch #1426

Open
shannynalayna opened this issue May 23, 2022 · 0 comments

Comments

@shannynalayna
Copy link
Contributor

shannynalayna commented May 23, 2022

Currently, Grid's TrackingBatch has batch_header and serialized_batch fields. The batch_header field is a String, populated by a Transact batch's header_signature. Grid's tracking batch serialized_batch field is taken from the transact batch's batch_header field. This naming is confusing as the batch_header field in the Grid struct is pulling the header_signature field from Transact, yet the Transact batch_header field is being used in the serialized_batch field in Grid. The batch_header field should be renamed to better reflect what this field is.

@shannynalayna shannynalayna created this issue from a note in Grid (Backlog) May 23, 2022
@ryjones ryjones removed this from Backlog in Grid Mar 23, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

No branches or pull requests

1 participant