feat: new architecture (turbo modules, fabric components) and unify return data from accounts sdk methods#132
Open
DanielOS7-TM wants to merge 41 commits intomainfrom
Open
feat: new architecture (turbo modules, fabric components) and unify return data from accounts sdk methods#132DanielOS7-TM wants to merge 41 commits intomainfrom
DanielOS7-TM wants to merge 41 commits intomainfrom
Conversation
* refactor: (wip)refactor all kotlin methods to not use runBlocking * refactor: remove return * refactor: native code to return the same methods and object data per platform * refactor: rename iOS memberInfo key names to match android * feat: android account sdk turbo module working * refactor: add GlobalEventEmitter back to TicketmasterIgnitePackage * feat: add config turbo module * feat: ios account sdk and config turbo module * feat: refactor global event emitter * fix: prebuilt module image on turbo modules * fix: prebuilt module image on turbo modules android
…ter-ignite into new-architecture
* refactor: update tickets sdk embedded RN native UI to a fabric component * chore: update comment * refactor: add tickets modal fabric component on ios * refactor: add tickets sdk fabric component android * chore: update build gradle * chore: rename accounts sdk files and classes
* feat: add retail sdk spec * refactor: add retail sdk turbo modules ios * refactor: tickets sdk modal fabric component to a turbo module * chore: remove tickets sdk modal component map * refactor: retail sdk present methods to turbo modules android * docs: add todo note * docs: add todo note
…ter-ignite into new-architecture
* chore: add secure entry spec and codegen config * refactor: secure entry ios native ui to a fabric component * refactor: secure entry from native ui to fabric component
…ter-ignite into new-architecture
Signed-off-by: Daniel Olugbade <110384307+DanielOS7-TM@users.noreply.github.com>
Signed-off-by: Daniel Olugbade <110384307+DanielOS7-TM@users.noreply.github.com>
…ter-ignite into new-architecture
…ter-ignite into new-architecture
Signed-off-by: Daniel Olugbade <110384307+DanielOS7-TM@users.noreply.github.com>
Signed-off-by: Daniel Olugbade <110384307+DanielOS7-TM@users.noreply.github.com>
…ter-ignite into new-architecture
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Migrates the library from the old React Native bridge architecture to the New Architecture (Fabric Components + TurboModules).
Accounts SDK — Replaced old
NativeModulesbridge with a TurboModule spec and codegen-backed implementations on iOS and Android. Unifiedlogin()return type to{ accessToken?: string }across both platforms. Extracted sharedfetchTokenData()helper on Android to deduplicate token priority logic.Retail SDK — Added TurboModule spec with native implementations for pre-purchase and purchase flows.
Tickets SDK — Converted
SecureEntryandTicketsSdkEmbeddedto Fabric native components with codegen specs, replacing the separate iOS/Android wrappers. ConvertedTicketsSdkModalto a TurboModule.Config — Migrated to a TurboModule with codegen spec.
Android — Updated
TicketmasterIgnitePackagefor TurboModule support.JS/TS naming — Renamed
SDK→Sdkin all JS/TS filenames, variables, and exports to follow standard JS casing conventions.Tests — Consolidated platform-specific test files, standardised mock variable prefixes, removed iOS/Android login branching now that both platforms return the same shape.
Dependencies — Updated
package.jsonfor New Architecture support.