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

F&F Issues. #2004

Closed
cakesoft-swati opened this issue Nov 9, 2020 · 10 comments
Closed

F&F Issues. #2004

cakesoft-swati opened this issue Nov 9, 2020 · 10 comments
Assignees
Labels
F&F Everything related to Friends and Family Reopen Status: fixed✅ It is fixed (PR merged)

Comments

@cakesoft-swati
Copy link

cakesoft-swati commented Nov 9, 2020

1.Unable to see the expiry timer on the F&F screen.
Screenshot_20201109-144543_hexa dev

2.Unable to see the wallet name on F&F screen even after confirm the F&F request(by skipping the contact name) but ,we can see the wallet name if we click on that contact .
20201109_153416

Screenshot_20201109-151409_hexa dev

@cakesoft-swati cakesoft-swati added F&F Everything related to Friends and Family 1.3.1 labels Nov 9, 2020
@cakesoft-swati cakesoft-swati changed the title Issues of F&F. F&F Issues. Nov 9, 2020
@CypherPoet
Copy link
Contributor

Regarding Unable to see the expiry timer on the F&F screen., this is related to the comment I left here.

Any clarity about the logic required for displaying the timer would be really useful (@AliMeer @antuz123 @cakesoft-nikhita @Parsh )

@CypherPoet
Copy link
Contributor

Regarding Unable to see the wallet name on F&F screen even after confirm the F&F request, is the issue that we want to show the wallet name even if other name properties exist? That is, does the wallet name take priority over everything else?

CypherPoet pushed a commit that referenced this issue Nov 10, 2020
@antuz123
Copy link
Member

Regarding Unable to see the wallet name on F&F screen even after confirm the F&F request, is the issue that we want to show the wallet name even if other name properties exist? That is, does the wallet name take priority over everything else?

Wallet name is used only when the Contact Name is not there (i.e. skip is used).

@antuz123
Copy link
Member

If the requester uses 'skip', the app does not know the Contact name or the Wallet name. That is where we use the temp string. Which needs to be replaced when the Wallet name is received back.

@cakesoft-swati
Copy link
Author

-Unable to see the changes on dev app version 1.3.1(161).
1.Unable to see the expiry timer on the F&F screen.
2.Unable to see the wallet name on F&F screen even after confirm the F&F request(by skipping the contact name) but ,we can see the wallet name if we click on that contact .

@cakesoft-swati cakesoft-swati added Reopen and removed Status: fixed✅ It is fixed (PR merged) labels Nov 12, 2020
CypherPoet pushed a commit that referenced this issue Nov 12, 2020
Connected to #2004.

This also removes the `CountDown` component, as it was duplicating
what we're already using from `react-native-countdown-component`.
@CypherPoet
Copy link
Contributor

Maybe I'm still not understanding how to reproduce this, but on the latest dev app version 1.3.1(161), the F&F screen appears to show the wallet name after confirming a request:

Screenshot_20201112-123912

@cakesoft-swati
Copy link
Author

-Still I'm not able to see the wallet name on F&F list after confirming the request.
-Tested on staging app version 1.3.1(162)
Screenshot_20201116-142523_hexa stage

Screenshot_20201116-142527_hexa stage

@cakesoft-swati cakesoft-swati removed the Status: fixed✅ It is fixed (PR merged) label Nov 16, 2020
@cakesoft-swati
Copy link
Author

I tried it by skipping the contact on both(sender and receivers) side.
I also tried it by skipping the contact on senders side.

@CypherPoet
Copy link
Contributor

CypherPoet commented Nov 17, 2020

I think this needs to be fixed by making a few changes on the backend. The UI components that display these names are falling back to the wallet name if a displayedName doesn't exist. However, displayedName is being derived from the first and last name being set for a contact item -- which is being set to "F&F Request" for contacts where name information can't be found (instead of null).

See the discussion here for more details

@AliMeer AliMeer assigned Parsh and unassigned CypherPoet Nov 17, 2020
@Parsh Parsh added the Status: fixed✅ It is fixed (PR merged) label Nov 17, 2020
@cakesoft-swati
Copy link
Author

-Verified this issue on dev app version 1.3.1(164)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
F&F Everything related to Friends and Family Reopen Status: fixed✅ It is fixed (PR merged)
Projects
None yet
Development

No branches or pull requests

5 participants