diff --git a/src/screens/StarterPack/Landing/index.tsx b/src/screens/StarterPack/Landing/index.tsx index e5202e9bc6..029e9ae79e 100644 --- a/src/screens/StarterPack/Landing/index.tsx +++ b/src/screens/StarterPack/Landing/index.tsx @@ -148,7 +148,7 @@ function LandingScreenInner({ Join Bluesky now - {joinedWeekCount && joinedWeekCount >= 25 && ( + {joinedWeekCount && joinedWeekCount >= 25 ? ( {joinedWeekCount} joined this week! - )} + ) : null} - {Boolean(starterPack.feeds?.length) && ( + {starterPack.feeds?.length ? ( Join Bluesky now to subscribe to these feeds @@ -182,9 +182,9 @@ function LandingScreenInner({ ))} - )} + ) : null} - {sampleProfiles?.length && ( + {Boolean(sampleProfiles?.length) && ( {feeds?.length ? ( @@ -203,12 +203,12 @@ function LandingScreenInner({ a.py_md, a.gap_xl, ]}> - {!!userSets.first?.length && ( + {userSets.first?.length ? ( - )} - {!!userSets.second?.length && ( + ) : null} + {userSets.second?.length ? ( - )} + ) : null} )} diff --git a/src/screens/StarterPack/Wizard/index.tsx b/src/screens/StarterPack/Wizard/index.tsx index 081d2c385c..d3c880e218 100644 --- a/src/screens/StarterPack/Wizard/index.tsx +++ b/src/screens/StarterPack/Wizard/index.tsx @@ -176,6 +176,12 @@ function WizardInner({ }, [setMinimalShellMode]), ) + const defaultName = _( + msg`${ + currentProfile?.displayName || `@${currentProfile?.handle}` + }'s Starter Pack`, + ) + const wizardUiStrings: Record< WizardStep, {header: string; nextBtn: string; subtitle?: string} @@ -208,13 +214,7 @@ function WizardInner({ const list = await agent.app.bsky.graph.list.create( {repo: currentAccount?.did}, { - name: - state.name ?? - _( - msg`${ - currentProfile?.displayName || `@${currentProfile?.handle}` - }'s Starter Pack`, - ), + name: state.name ?? defaultName, description: state.description, descriptionFacets: [], avatar: undefined, @@ -309,8 +309,8 @@ function WizardInner({ collection: 'app.bsky.graph.starterpack', rkey, record: { - name: state.name ?? '', - description: state.description ?? '', + name: state.name ?? defaultName, + description: state.description, descriptionFacets: [], list: list?.uri, feeds: state.feeds.map(f => ({ @@ -343,8 +343,8 @@ function WizardInner({ validate: false, }, { - name: state.name ?? '', - description: state.description ?? '', + name: state.name ?? defaultName, + description: state.description, descriptionFacets: [], list: list?.uri, feeds: state.feeds.map(f => ({ @@ -395,14 +395,7 @@ function WizardInner({ } const onNext = () => { - if (state.currentStep === 'Details' && !state.name) { - dispatch({ - type: 'SetName', - name: _( - msg`${currentProfile?.displayName || currentProfile?.handle}'s`, - ), - }) - } else if (state.currentStep === 'Feeds') { + if (state.currentStep === 'Feeds') { submit() return }