cleanup landing

src/screens/StarterPack/Landing/index.tsx

@@ -148,7 +148,7 @@ function LandingScreenInner({
               <Trans>Join Bluesky now</Trans>
             </ButtonText>
           </Button>
-          {joinedWeekCount && joinedWeekCount >= 25 && (
+          {joinedWeekCount && joinedWeekCount >= 25 ? (
             <Text
               style={[
                 a.font_bold,
@@ -158,10 +158,10 @@ function LandingScreenInner({
               ]}>
               {joinedWeekCount} joined this week!
             </Text>
-          )}
+          ) : null}
           <Divider />
           <View style={[a.gap_3xl]}>
-            {Boolean(starterPack.feeds?.length) && (
+            {starterPack.feeds?.length ? (
               <View style={[a.gap_md]}>
                 <Text style={[a.font_bold, a.text_lg]}>
                   Join Bluesky now to subscribe to these feeds
@@ -182,9 +182,9 @@ function LandingScreenInner({
                   ))}
                 </View>
               </View>
-            )}
+            ) : null}
 
-            {sampleProfiles?.length && (
+            {Boolean(sampleProfiles?.length) && (
               <View style={[a.gap_md]}>
                 <Text style={[a.font_bold, a.text_lg]}>
                   {feeds?.length ? (
@@ -203,12 +203,12 @@ function LandingScreenInner({
                     a.py_md,
                     a.gap_xl,
                   ]}>
-                  {!!userSets.first?.length && (
+                  {userSets.first?.length ? (
                     <ProfilesSet profiles={userSets.first} />
-                  )}
+                  ) : null}
-                  {!!userSets.second?.length && (
+                  {userSets.second?.length ? (
                     <ProfilesSet profiles={userSets.second} />
-                  )}
+                  ) : null}
                 </View>
               </View>
             )}

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:
+        name: state.name ?? defaultName,
-          state.name ??
-          _(
-            msg`${
-              currentProfile?.displayName || `@${currentProfile?.handle}`
-            }'s Starter Pack`,
-          ),
         description: state.description,
         descriptionFacets: [],
         avatar: undefined,
@@ -309,8 +309,8 @@ function WizardInner({
           collection: 'app.bsky.graph.starterpack',
           rkey,
           record: {
-            name: state.name ?? '',
+            name: state.name ?? defaultName,
-            description: state.description ?? '',
+            description: state.description,
             descriptionFacets: [],
             list: list?.uri,
             feeds: state.feeds.map(f => ({
@@ -343,8 +343,8 @@ function WizardInner({
             validate: false,
           },
           {
-            name: state.name ?? '',
+            name: state.name ?? defaultName,
-            description: state.description ?? '',
+            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) {
+    if (state.currentStep === 'Feeds') {
-      dispatch({
-        type: 'SetName',
-        name: _(
-          msg`${currentProfile?.displayName || currentProfile?.handle}'s`,
-        ),
-      })
-    } else if (state.currentStep === 'Feeds') {
       submit()
       return
     }