Skip to content
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

Implement transaction request #21

Merged
merged 24 commits into from
Nov 15, 2018
Merged

Conversation

ripzery
Copy link

@ripzery ripzery commented Oct 24, 2018

Issue/Task Number: #20

Closes #20

Overview

This PRs implement 2 things:

1. Transaction request flow

The detail is here:

  • The app will generate the transaction request(for receive and send) every time after the primary token is set, then save ids to the share preference.

  • The app will listen to the user socket event every time the authenticated user opens the app.

  • The app will stop listening to the user socket event every time the app is closed or log out.

  • For the transaction request consumption type send, the app will show the confirmation page to confirm a transaction whether approve or reject when it is consumed.

  • For the transaction request consumption type receive, the app will just update the balance of the app.

2. Tests

Tests are cover the following use-cases:

  • Set the primary token
  • Receive an event transaction request consumption type send
  • Receive an event transaction request consumption type receive

Note: The gif will come later

@ripzery ripzery added the enhancement New feature or request label Oct 24, 2018
@ripzery ripzery requested review from T-Dnzt and mederic-p October 24, 2018 04:22
@ripzery ripzery changed the title 20 implement transaction request Implement transaction request Oct 27, 2018
@ripzery ripzery merged commit 01a2202 into master Nov 15, 2018
@ripzery ripzery deleted the 20-implement-transaction-request branch November 15, 2018 05:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implement transaction request
3 participants