Paths

Table of Contentst

Differential D10839

[lib/native] lift staff provider into lib
ClosedPublic
Actions

Authored by ginsu on Jan 26 2024, 8:18 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Thu, Oct 31, 12:25 PM

	Unknown Object (File)
	Thu, Oct 31, 12:25 PM

	Unknown Object (File)
	Thu, Oct 31, 12:25 PM

	Unknown Object (File)
	Thu, Oct 31, 12:24 PM

	Unknown Object (File)
	Thu, Oct 31, 12:21 PM

	Unknown Object (File)
	Oct 6 2024, 7:39 AM

	Unknown Object (File)
	Oct 6 2024, 7:15 AM

	Unknown Object (File)
	Oct 6 2024, 6:41 AM

View All 50 Files

Subscribers

Details

Reviewers

atul
inka

Commits

rCOMM6025a259cd6f: [lib/native] lift staff provider into lib

Summary

As I was building the build info page for web, I noticed that the hasStaffUserLoggedIn only lives in native. To get this field into web we should lift the staff context provider into lib so that it can be used on both platforms. This diff lifts the staff provider into lib so that it can be used in both web and native

Depends on D10825

Test Plan

flow + confirmed that there were no regressions with native

Diff Detail

Repository

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

ginsu created this revision.Jan 26 2024, 8:18 AM

Herald added subscribers: tomek, ashoat. · View Herald TranscriptJan 26 2024, 8:18 AM

ginsu edited the summary of this revision. (Show Details)Jan 26 2024, 8:19 AM

ginsu added reviewers: atul, inka.

ginsu added a parent revision: D10825: [native] consolidate staff context provider into one file.

ginsu added a child revision: D10840: [web] introduce build info panel.Jan 26 2024, 8:31 AM

Harbormaster completed remote builds in B26193: Diff 36175.Jan 26 2024, 8:39 AM

ginsu requested review of this revision.Jan 26 2024, 8:39 AM

confirmed that there were no regressions with native

Ideally we'd have a more specific test plan that goes into a few scenarios that were checked

This revision is now accepted and ready to land.Jan 29 2024, 10:01 PM

rebase

Harbormaster completed remote builds in B26376: Diff 36466.Jan 31 2024, 1:04 PM

Closed by commit rCOMM6025a259cd6f: [lib/native] lift staff provider into lib (authored by ginsu). · Explain WhyJan 31 2024, 1:08 PM

This revision was automatically updated to reflect the committed changes.

ginsu added a commit: rCOMM6025a259cd6f: [lib/native] lift staff provider into lib.

Revision Contents
Changeset List

Path

Size

	lib/	components/
	native/	staff/

	staff-provider.react.js
	staff-context.provider.react.js

2 lines

native/

account/

logged-out-staff-info.react.js

3 lines

data/

sqlite-data-handler.js

2 lines

profile/

build-info.react.js

2 lines

2 lines

staff/

staff-context.provider.react.js

Diff 36466

lib/components/staff-provider.react.js

Loading...

native/account/logged-out-staff-info.react.js

Loading...

native/data/sqlite-data-handler.js

Loading...

native/profile/build-info.react.js

Loading...

native/root.react.js

Loading...

native/staff/staff-context.provider.react.js

Loading...