[native] remove exitingCallback
ClosedPublic
Actions

Authored by varun on Tue, Nov 12, 1:51 PM.

Tags

None

Referenced Files

	F3329418: D13920.id.diff
	Wed, Nov 20, 4:56 PM

	F3329076: D13920.id45777.diff
	Wed, Nov 20, 4:37 PM

	F3328934: D13920.id45802.diff
	Wed, Nov 20, 4:19 PM

	Unknown Object (File)
	Tue, Nov 19, 5:12 PM

	Unknown Object (File)
	Tue, Nov 19, 3:10 PM

	Unknown Object (File)
	Mon, Nov 18, 7:23 PM

	Unknown Object (File)
	Mon, Nov 18, 9:54 AM

	Unknown Object (File)
	Mon, Nov 18, 9:01 AM

View All 14 Files

Subscribers

None

Details

Reviewers

ashoat
tomek

Commits

rCOMM485956d0276d: [native] remove exitingCallback

Summary

we should just call navigation.goBack when we reach the end of the ordered list of tips. there's no need for the exitingCallback

Depends on D13917

Test Plan

confirmed that when i pressed ok on the final tip, i was backed out from the backdrop

Diff Detail

Repository

rCOMM Comm

Branch

master

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

varun created this revision.Tue, Nov 12, 1:51 PM

varun added 1 blocking reviewer(s): tomek.Tue, Nov 12, 2:05 PM

Harbormaster completed remote builds in B32637: Diff 45775.Tue, Nov 12, 2:59 PM

varun requested review of this revision.Tue, Nov 12, 2:59 PM

tomek accepted this revision.Wed, Nov 13, 5:20 AM

This revision is now accepted and ready to land.Wed, Nov 13, 5:20 AM

Closed by commit rCOMM485956d0276d: [native] remove exitingCallback (authored by varun). · Explain WhyWed, Nov 13, 11:00 AM

This revision was automatically updated to reflect the committed changes.

varun added a commit: rCOMM485956d0276d: [native] remove exitingCallback.

Revision Contents
Changeset List

Path

Size

native/

components/

nux-tips-context.react.js

5 lines

tooltip/

nux-tips-overlay.react.js

8 lines

Diff 45775

View Options

native/components/nux-tips-context.react.js