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

Integrate ios local mobile terms of service accept callback #771

Merged
merged 1 commit into from
Aug 8, 2024

Conversation

tim-lin-bbpos
Copy link
Collaborator

@tim-lin-bbpos tim-lin-bbpos commented Aug 1, 2024

Summary

Add callback event when user accept the terms of service of using tap to play on iOS side.

Motivation

Part of the request from https://bugs.bbpos.com/projects/SDK/issues/SDK-189
iOS ref: https://stripe.dev/stripe-terminal-ios/docs/Protocols/SCPLocalMobileReaderDelegate.html#/c:objc(pl)SCPLocalMobileReaderDelegate(im)localMobileReaderDidAcceptTermsOfService:

Testing

  • I tested this manually
  • I added automated tests

Documentation

Select one:

  • I have added relevant documentation for my changes.
  • This PR does not result in any developer-facing changes.

@tim-lin-bbpos tim-lin-bbpos self-assigned this Aug 1, 2024
@tim-lin-bbpos tim-lin-bbpos changed the title Integrate ios local mobile terms of service accept callback [WIP] Integrate ios local mobile terms of service accept callback Aug 1, 2024
@tim-lin-bbpos tim-lin-bbpos changed the title [WIP] Integrate ios local mobile terms of service accept callback Integrate ios local mobile terms of service accept callback Aug 1, 2024
Copy link

@gliu-stripe gliu-stripe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm! Minor nit but non-blocking

@@ -94,6 +94,9 @@ export default function DiscoverReadersScreen() {
onDidReportAvailableUpdate: (update) => {
Alert.alert('New update is available', update.deviceSoftwareVersion);
},
onDidAcceptTermsOfService: () => {
Alert.alert('Accept terms of Service');

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: Should this be "Accepted Terms of Service"?
not too important since it's just the dev app, but just fyi that this callback fires after acceptance

@nazli-stripe nazli-stripe merged commit ef8d354 into main Aug 8, 2024
3 checks passed
@tim-lin-bbpos tim-lin-bbpos deleted the bbpos/local-mobile-terms-of-service branch August 12, 2024 07:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants