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

Paginated RPC pay.operations #4394

Merged
merged 4 commits into from
Jul 4, 2019
Merged

Paginated RPC pay.operations #4394

merged 4 commits into from
Jul 4, 2019

Conversation

jiivan
Copy link
Contributor

@jiivan jiivan commented Jun 27, 2019

Part of #4172
Fixes: #2402
Fixes: #3971

@jiivan jiivan force-pushed the task_payment_rpc branch 2 times, most recently from 8ce3b4d to 7c23c3c Compare July 3, 2019 10:28
@jiivan jiivan force-pushed the task_payment_rpc branch from 7c23c3c to 9a7caf4 Compare July 3, 2019 10:44
@jiivan jiivan requested a review from etam July 3, 2019 11:15
@jiivan jiivan marked this pull request as ready for review July 3, 2019 11:15
@etam
Copy link
Contributor

etam commented Jul 3, 2019

Is it related to any of #3971, #3970, #2402?

@jiivan
Copy link
Contributor Author

jiivan commented Jul 3, 2019

Is it related to any of #3971, #3970, #2402?

It will somehow fix #3971 and #2402 but not #3970

@jiivan jiivan force-pushed the task_payment_rpc branch from 8fe47df to 5eae753 Compare July 4, 2019 10:09
Copy link
Contributor

@etam etam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@@ -484,6 +485,8 @@ def add_payment(self, processed_ts: int):
wallet_operation__direction= # noqa
model.WalletOperation.DIRECTION.outgoing,
created_date=timestamp_to_datetime(processed_ts),
wallet_operation__status= # noqa
model.WalletOperation.STATUS.awaiting,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this could be a one line, like above

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(I mean in previous change, in line 66)

@codecov
Copy link

codecov bot commented Jul 4, 2019

Codecov Report

Merging #4394 into develop will decrease coverage by 1.67%.
The diff coverage is 78.57%.

@@             Coverage Diff             @@
##           develop    #4394      +/-   ##
===========================================
- Coverage    88.42%   86.74%   -1.68%     
===========================================
  Files          224      224              
  Lines        19841    19826      -15     
===========================================
- Hits         17544    17199     -345     
- Misses        2297     2627     +330

@jiivan jiivan merged commit 608aae0 into develop Jul 4, 2019
@jiivan jiivan deleted the task_payment_rpc branch July 4, 2019 11:39
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implement pagination for outgoing payments Implement pagination for getting incomes
2 participants