-
Notifications
You must be signed in to change notification settings - Fork 17
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
chore(dashpay): restoring Contacts flow #584
chore(dashpay): restoring Contacts flow #584
Conversation
@@ -24,7 +24,6 @@ extern NSNotificationName const DWDashPayAvailabilityStatusUpdatedNotification; | |||
@protocol DWDashPayReadyProtocol <NSObject> | |||
|
|||
@property (readonly, nonatomic, assign) BOOL isDashPayReady; | |||
@property (readonly, nonatomic, assign) BOOL isDashPayReadyMainSuggestion; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not needed atm. Removed to keep the DWDashPayReadyProtocol
footprint low.
@@ -42,7 +43,7 @@ NS_ASSUME_NONNULL_BEGIN | |||
- (void)homeModelWantToReloadShortcuts:(id<DWHomeProtocol>)model; | |||
@end | |||
|
|||
@protocol DWHomeProtocol <DWTxDisplayModeProtocol> | |||
@protocol DWHomeProtocol <DWTxDisplayModeProtocol, DWDashPayReadyProtocol> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Contacts VC requires DWDashPayReadyProtocol
which was implemented by the home model before. Perhaps there is a path to refactor this in the future stories.
@@ -24,5 +24,6 @@ | |||
|
|||
NSInteger DW_MIN_USERNAME_LENGTH = 3; | |||
NSInteger DW_MAX_USERNAME_LENGTH = 24; | |||
BOOL MOCK_DASHPAY = YES; // TODO: remove once Platform is available |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I figure it's better to reference a single flag for any mocking code. Will be easier to turn off and remove once the platform is available.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
Issue being fixed or feature implemented
As a part of the "Bringing back DashPay" initiative, the contacts entry point in the tabbar should be restored.
What was done?
dashPayModel
to sync with theHomeHeaderView
How Has This Been Tested?
Breaking Changes
Checklist:
For repository code-owners and collaborators only