mobile-wallet icon indicating copy to clipboard operation
mobile-wallet copied to clipboard

fix: updated the PR- feat : Implemented details on successful transaction

Open alphaNewrex opened this issue 2 years ago • 3 comments

Issue Fix

Fixes merge issues in PR #428 and Fixes Issue #425

Description

Added a transaction receipt when a transaction is successful. this can be used to send the screenshot of the amount of money sent to anyone in order to confirm the money dispatched from our side.

  • [x] Apply the AndroidStyle.xml style template to your code in Android Studio.

  • [x] Run the unit tests with ./gradlew check to make sure you didn't break anything

  • [x] If you have multiple commits please combine them into one commit by squashing them.

alphaNewrex avatar Apr 11 '22 10:04 alphaNewrex

@devansh-299 look into this PR and please review it.

alphaNewrex avatar Apr 11 '22 10:04 alphaNewrex

@devansh-299 look into this PR and please review it.

@alphaNewrex UI is not up to the mark, please make the following changes:

  • The heading text should be changed to Transaction Successful
  • The views should be arranged in the following verticle fashion:
    1. Transaction ID as Transaction Id: <transaction id>
    2. The receiver as Transferred To: <receiver name>
    3. Amount as Amount: <currency><amount>

devansh-299 avatar Apr 12 '22 12:04 devansh-299

@devansh-299 I have updated the

UI and made it Align Vertical. A sample for the same is attached below. Since the demo account credentials didn't have any transactions associated with them. I'm attaching a screenshot from Design Tab. The Text for each will be updated by the code. Screenshot 2022-04-15 013228

alphaNewrex avatar Apr 14 '22 20:04 alphaNewrex