Goodsdks migration setup#9
Merged
Merged
Conversation
Contributor
Author
|
@copilot can you review and match it against the GoodSDKs related pull-request: GoodDollar/GoodSDKs#42 and update this pull-request description with a clear and concise explanation what the changes propose |
Done. The PR description has been updated to focus only on the GoodSDKs migration changes ( |
9 tasks
…nto goodsdks-migration-setup
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
Sets up the
@goodwidget/citizen-claim-widgetmigration scaffolding, aligned with GoodSDKs#42.GoodSDKs#42 adds a capability manifest (
citizenSdkCapabilities) to@goodsdks/citizen-sdkthat formally documents what the SDK can do — runtime needs, supported chains, and the full set of read/write operations. This PR wires GoodWidget up to that capability model by introducing the newpackages/citizen-claim-widgetpackage.Changes —
packages/citizen-claim-widgetA new package that bridges
@goodsdks/citizen-sdkinto the GoodWidget architecture. It contains only migration piping for now (no UI) and is the target for the future full citizen-claim widget.adapterContract.ts— TypeScript types defining the widget's state machine and host API:CitizenClaimWidgetStatus— all possible widget states:loading,not_connected,not_whitelisted,eligible,already_claimed,claiming,success,errorCitizenClaimWidgetAdapterState/CitizenClaimWidgetAdapterActions— the shape any SDK adapter must satisfyCitizenClaimWidgetProps— host-facing props:provider,environment,clientFactory,onClaimSuccess,onClaimErrorintegration.ts— Links the widget to the SDK capability manifest from GoodSDKs#42:whitelistStatus,claimStatus,claimEntitlement,startVerification,claimcitizenSdkCapabilitiesby name so future tooling can validate the integration against the SDKindex.ts— Re-exports both the integration descriptor and all adapter contract types.Relationship to GoodSDKs#42
SdkCapabilitySummary/SdkCapabilityOperationtypescitizenClaimIntegrationshapecitizenSdkCapabilitiesmanifestintegration.ts(capabilitySource: 'citizenSdkCapabilities')whitelistStatus,identityExpiry,claimStatus,claimEntitlementusesstartVerification,claimuseschainsThe existing
@goodwidget/claim-widget(theme/demo widget) is not changed by this migration.