-
Notifications
You must be signed in to change notification settings - Fork 15
DTSERWFOUR-306 Tag/1.0.0 beta08 #235
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
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Collaborator
vwagh-hw
commented
Jan 28, 2021
- Added support for update Transfer method
- Enhancements
* HW-52538 add support to PayPal account
*HW-51711 Get user details
Extra ids added
* HW-53054: Re-organize package structure
* HW-53384: Update UI group id
#210) * DTSERWFOUR-167 - Add localization string for 33 languages for commonui, receiptui, transfermethodrepository, transfermethodui and transferui * DTSERWFOUR-167 - Addressed review comments * DTSERWFOUR-167 - Fix for Traditional Chinese. Co-authored-by: Rita Law <51347548+malaw-hw@users.noreply.github.com>
#211) * DTSERWFOUR-139 - On navigation refreshed the home, create transfer & transaction * DTSERWFOUR -139 - fixed transfer source bug * `DTSERWFOUR-139 - Addressed review comments` Co-authored-by: Rita Law <51347548+malaw-hw@users.noreply.github.com> Co-authored-by: ssabeerahamed <ssabeerahamed@paypal.com>
#203) * DTSERWFOUR-147-Alignment-Bug-fixes - Incorrect Spacing between Subheader and content * task/DTSERWFOUR-147-Alignment-Bug-fixes - Fixed header top * task/DTSERWFOUR-147-Alignment-Bug-fixes - Transfer Method & Add Transfer Method * task/DTSERWFOUR-147-Alignment-Bug-fixes Align the headers to the same Height * task/DTSERWFOUR-147-Alignment-Bug-fixes Review comments updated * task/DTSERWFOUR-147-Alignment-Bug-fixes Transfer fund alignment issue fixed * task/DTSERWFOUR-147-Alignment-Bug-fixes Transfer Method & Add Transfer Method * task/DTSERWFOUR-147-Alignment-Bug-fixes Review comments updated * task/DTSERWFOUR-147-Alignment-Bug-fixes Review comments updated Co-authored-by: Rita Law <51347548+malaw-hw@users.noreply.github.com>
Co-authored-by: Rita Law <51347548+malaw-hw@users.noreply.github.com>
* bug/DTSERWFOUR-170-Remaining-Screen-Updates changes updated * bug/DTSERWFOUR-170-Remaining-Screen-Updates - Confirmation screen update * bug/DTSERWFOUR-170-Remaining-Screen-Updates logic updates * DTSERWFOUR -170 - formatted currency in details screen. * DTSERWFOUR -170 - Fixed UI Test. * DTSERWFOUR -170 - Fixed invalid error. * DTSERWFOUR -170 - resolve merge conflict * DTSERWFOUR -170 - fixed Currency Symbol in confirm screen. * bug/DTSERWFOUR-170-Remaining-Screen-Updates Fixed JacocoTestcoverage Report * DTSERWFOUR -170 - fixed TC failure. * DTSERWFOUR -170 - fixed right currency symbols. * DTSERWFOUR -170 addressed review comments. * bug/DTSERWFOUR-170-Remaining-Screen-Updates Code review updates Co-authored-by: Rita Law <51347548+malaw-hw@users.noreply.github.com> Co-authored-by: ssabeerahamed <ssabeerahamed@paypal.com> Co-authored-by: Rajkumar Balakrishnasamy <71116144+rajkumar-balakrishnasamy@users.noreply.github.com>
# Conflicts: # build.gradle
* DTSERWFOUR -184 support paper check in UI SDK. * DTSERWFOUR -184 support paper check. * DTSERWFOUR -184 support paper check in UI SDK. * DTSERWFOUR -184 added UI test cause. * DTSERWFOUR -184 added UI test cause. * DTSERWFOUR -184 added UI test cause. * DTSERWFOUR -184 addressed PR comments. * DTSERWFOUR -184 addressed PR comments. * DTSERWFOUR -184 issues fixed. * DTSERWFOUR -184- fixed toolbar issue. * fix assertions Co-authored-by: Rita Law <malaw@paypal.com> Co-authored-by: Rita Law <51347548+malaw-hw@users.noreply.github.com>
#218) * bug/DTSERWFOUR-30-transactions-Screen-Minimize-header-Transfer-Screen-fix - Issue fixes * bug/DTSERWFOUR-30-transactions-Screen-Minimize-header-Transfer-Screen-fix Issue fixed * bug/DTSERWFOUR-30-transactions-Screen-Minimize-header-Transfer-Screen-fix - Code review comments Co-authored-by: Rita Law <51347548+malaw-hw@users.noreply.github.com>
* DTSERWFOUR -189 wrap labels. * DTSERWFOUR -189 wrap labels. * DTSERWFOUR -189 wrap labels. * DTSERWFOUR -189 wrap labels to next line. * DTSERWFOUR -189 address review comments. Co-authored-by: Rita Law <51347548+malaw-hw@users.noreply.github.com>
* task/DTSERWFOUR-142-wrap-truncated-headers - fixes * task/DTSERWFOUR-142-wrap-truncated-headers fixes * DTSERWFOUR-142 - Fixed minor scenarios * DTSERWFOUR-142 - Reverted to wrap content for select transfer screen. * DTSERWFOUR-142 - Reverted to style change * DTSERWFOUR-142 - Optimized the code. * DTSERWFOUR-142 - Fixed the observation. * task/DTSERWFOUR-142-wrap-truncated-headers - fixes Co-authored-by: Rita Law <51347548+malaw-hw@users.noreply.github.com> Co-authored-by: Rajkumar <rbalakrishnasamy@paypal.com>
…e devices (#224) * DTSERWFOUR-175-TRANSACTION-section-label-is-not-displayed-in-some-devices * DTSERWFOUR-175-TRANSACTION-section-label-is-not-displayed-in-some-devices * DTSERWFOUR-175-TRANSACTION-section-label-is-not-displayed-in-some-devices * fixed testListPrepaidCardReceipt_checkDateTextOnLocaleChange * Since static header got refactored to minimize based on scrolling. Now deleted unreferenced code. Co-authored-by: Rita Law <malaw@paypal.com> Co-authored-by: ssidigum <ssidigum@paypal.com>
* Fixed failed test cases * Fixed failed test cases Co-authored-by: Rita Law <51347548+malaw-hw@users.noreply.github.com>
* task/DTSERWFOUR-123-Transfer-Funds-Zero-Funds * task/DTSERWFOUR-123-Transfer-Funds-Zero-Funds commented failed tests * task/DTSERWFOUR-123-Transfer-Funds-Zero-Funds * task/DTSERWFOUR-123-Transfer-Funds-Zero-Funds Failed test cases fixed * Added 2 scenarios UI tests for not Available fund * task/DTSERWFOUR-123-Transfer-Funds-Zero-Funds Co-authored-by: Rita Law <malaw@paypal.com> Co-authored-by: Rita Law <51347548+malaw-hw@users.noreply.github.com>
* DTSERWFOUR -210 - Update transfer method information * DTSERWFOUR-210- added Unit test * DTSERWFOUR-210- update widget * DTSERWFOUR-210- update UI-SDK-Repo * DTSERWFOUR-211 - Fetch transfer method information and display for edits. * update unit Test causes * DTSERWFOUR-211 - Updated the code. * update unit Test causes * update failed unit Test causes * update api call for Update * DTSERWFOUR-211 - Handled the update code for edit. * DTSERWFOUR-211 - Fixed minor Bugs. * fixed failed test causes * Update widget * DTSERWFOUR-211 - Resolved the merge conflicts * Update widget for masked field * DTSERWFOUR-211 - Fixed the Debit card number display issue. * DTSERWFOUR-211 - Fixed the Expiry Date display issue. * DTSERWFOUR-212 - Add edit button in the transfer methods list. * Address review comments. * Address review comments. * DTSERWFOUR-212 - Fixed the observations. * DTSERWFOUR-212 - Fixed the observations. * Fixed blocking API call if no changes in edit. * Implemented new class file to update transfer methods(Update Bank Account) and added new respective JSON mock response files. * DTSERWFOUR-212 - Fixed the key listener observation. * Updated widget * DTSERWFOUR-212 - Add edit button in the transfer methods list. (#227) * DTSERWFOUR-212 - Add edit button in the transfer methods list. * DTSERWFOUR-212 - Fixed the observations. * Fixed blocking API call if no changes in edit. * Implemented new class file to update transfer methods(Update Bank Account) and added new respective JSON mock response files. * DTSERWFOUR-212 - Fixed the key listener observation. Co-authored-by: ssabeerahamed <ssabeerahamed@paypal.com> Co-authored-by: ssidigum <ssidigum@paypal.com> * DTSERWFOUR-211 - Fetch transfer method information and display for ed… (#225) * DTSERWFOUR-211 - Fetch transfer method information and display for edits. * DTSERWFOUR-211 - Updated the code. * DTSERWFOUR-211 - Handled the update code for edit. * DTSERWFOUR-211 - Fixed minor Bugs. * DTSERWFOUR-211 - Resolved the merge conflicts * Update widget for masked field * DTSERWFOUR-211 - Fixed the Debit card number display issue. * DTSERWFOUR-211 - Fixed the Expiry Date display issue. * DTSERWFOUR-212 - Fixed the observations. * DTSERWFOUR-212 - Add edit button in the transfer methods list. (#227) * DTSERWFOUR-212 - Add edit button in the transfer methods list. * DTSERWFOUR-212 - Fixed the observations. * Fixed blocking API call if no changes in edit. * Implemented new class file to update transfer methods(Update Bank Account) and added new respective JSON mock response files. * DTSERWFOUR-212 - Fixed the key listener observation. Co-authored-by: ssabeerahamed <ssabeerahamed@paypal.com> Co-authored-by: ssidigum <ssidigum@paypal.com> Co-authored-by: ssabeerahamed <ssabeerahamed@paypal.com> Co-authored-by: ssidigum <ssidigum@paypal.com> * Addressed review comments. * Addressed review comments. * DTSERWFOUR-212 - Fixed test cases. * DTSERWFOUR-212 - Updated test cases. Co-authored-by: Rajkumar <rbalakrishnasamy@paypal.com> Co-authored-by: ssidigum <ssidigum@paypal.com> Co-authored-by: Rajkumar Balakrishnasamy <71116144+rajkumar-balakrishnasamy@users.noreply.github.com>
…nmo, papercheck, paypal, card. (#230)
* Code updated as VenmoTest * 1. fixed testUpdateTransferMethodFragment_verifyUpdateVenmoTransferMethod() 2. Assert both icons * 1. fixed testUpdateTransferMethodFragment_verifyUpdateVenmoTransferMethod() 2. Assert both icons
* task/DTSERWFOUR-280-Update-content-strings modified strings * task/DTSERWFOUR-280-Update-content-strings modified strings
* Enhancements
malaw-hw
approved these changes
Jan 28, 2021
ddhar-hw
approved these changes
Jan 28, 2021
vwagh-hw
added a commit
that referenced
this pull request
Feb 2, 2021
* DTSERWFOUR-306 Tag/1.0.0 beta08 (#235) * Added support for update Transfer method * Enhancements * Updating README.md * updated version to 1.0.0-beta09-SNAPSHOT Co-authored-by: Vikas Wagh <37385596+vwagh-hw@users.noreply.github.com> Co-authored-by: Vikas Wagh <cwagh@paypal.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.