Skip to content

Commit 037aa18

Browse files
committed
PR feedback
1 parent 153ad7e commit 037aa18

File tree

3 files changed

+10
-24
lines changed

3 files changed

+10
-24
lines changed

lang/en.lyaml

-23
Original file line numberDiff line numberDiff line change
@@ -651,33 +651,10 @@ en:
651651
componentsThatWillNotBeMigrated: "[NOTE] These component types are not yet supported for migration but will be available later: {{ components }}"
652652
errors:
653653
noAppsForProject: "No apps associated with project {{ projectName }}"
654-
noAppsEligible: "No apps in account {{ accountId }} are currently migratable"
655654
noAccountConfig: "There is no account associated with {{ accountId }} in the config file. Please choose another account and try again, or authenticate {{ accountId }} using {{ authCommand }}."
656-
invalidAccountTypeTitle: "{{#bold}}Developer account not targeted{{/bold}}"
657-
invalidAccountTypeDescription: "Only public apps created in a developer account can be converted to a project component. Select a connected developer account with {{useCommand}} or {{authCommand}} and try again."
658655
projectAlreadyExists: "A project with name {{ projectName }} already exists. Please choose another name."
659656
invalidApp: "Could not migrate appId {{ appId }}. This app cannot be migrated at this time. Please choose another public app."
660-
appWithAppIdNotFound: "Could not find an app with the id {{ appId }} "
661657
migrationFailed: 'Migration Failed'
662-
prompt:
663-
chooseApp: 'Which app would you like to migrate?'
664-
inputName: '[--name] What would you like to name the project?'
665-
inputDest: '[--dest] Where would you like to save the project?'
666-
uidForComponent: 'What UID would you like to use for {{ componentName }}?'
667-
proceed: 'Would you like to proceed?'
668-
spinners:
669-
beginningMigration: "Beginning migration"
670-
unableToStartMigration: "Unable to begin migration"
671-
finishingMigration: "Wrapping up migration"
672-
migrationComplete: "Migration completed"
673-
migrationFailed: "Migration failed"
674-
downloadingProjectContents: "Downloading migrated project files"
675-
downloadingProjectContentsComplete: "Migrated project files downloaded"
676-
downloadingProjectContentsFailed: "Unable to download migrated project files"
677-
copyingProjectFiles: "Copying migrated project files"
678-
copyingProjectFilesComplete: "Migrated project files copied"
679-
copyingProjectFilesFailed: "Unable to copy migrated project files"
680-
681658
cloneApp:
682659
describe: "Clone a public app using the projects framework."
683660
examples:

lang/en.ts

+2-1
Original file line numberDiff line numberDiff line change
@@ -3455,7 +3455,8 @@ export const lib = {
34553455
`Only public apps created in a developer account can be converted to a project component. Select a connected developer account with ${useCommand} or ${authCommand} and try again.`,
34563456
appWithAppIdNotFound: appId =>
34573457
`Could not find an app with the id ${appId} `,
3458-
noAppsForProject: 'No apps associated with project {{ projectName }}',
3458+
noAppsForProject: (projectName: string) =>
3459+
`No apps associated with project ${projectName}`,
34593460
migrationFailed: 'Migration Failed',
34603461
},
34613462
prompt: {

lib/app/migrate.ts

+8
Original file line numberDiff line numberDiff line change
@@ -89,6 +89,14 @@ async function fetchMigrationApps(
8989
throw new Error(lib.migrate.errors.project.multipleApps);
9090
}
9191

92+
if (allApps.length === 0 && projectConfig) {
93+
throw new Error(
94+
lib.migrate.errors.noAppsForProject(
95+
projectConfig?.projectConfig?.name || ''
96+
)
97+
);
98+
}
99+
92100
if (
93101
allApps.length === 0 ||
94102
filteredUnmigratableApps.length === allApps.length

0 commit comments

Comments
 (0)