-
Notifications
You must be signed in to change notification settings - Fork 103
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
feat!: Add dates as a top level field to credit batches (#393) #412
Conversation
Codecov Report
@@ Coverage Diff @@
## master #412 +/- ##
==========================================
+ Coverage 55.10% 56.09% +0.99%
==========================================
Files 33 59 +26
Lines 931 3742 +2811
==========================================
+ Hits 513 2099 +1586
- Misses 376 1372 +996
- Partials 42 271 +229
Flags with carried forward coverage won't be shown. Click here to find out more. |
410844d
to
605c3f4
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.
LGTM
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.
approving, but left 2 suggestions which will improve the code maintenance.
@@ -30,6 +30,14 @@ message EventCreateBatch { | |||
|
|||
// total_amount is the total number of credits in the credit batch. | |||
string total_amount = 4 [ (gogoproto.moretags) = "yaml:\"total_amount\"" ]; | |||
|
|||
// start_date is the beginning of the period during which this credit batch | |||
// was quantified and verified. |
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.
instead of copying, how about saying:
start data is the MsgCreateBatchRequest.start_date value
Same for end_date.
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.
lgtm
Description
Add
start_date
andend_date
fields toBatchInfo
and associatedMsgCreateBatchRequest
. These dates track the period during which the credit batch was quantified and verified.Closes: #393
Author Checklist
All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.
I have...
!
to the type prefix if API or client breaking changeCHANGELOG.md
Reviewers Checklist
All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items.
I have...
!
in the type prefix if API or client breaking change