Skip to content

Commit

Permalink
fix(cli): virtual tilesets are required for importing into dynamo (#3022
Browse files Browse the repository at this point in the history
)

#### Motivation

Virtual tile sets such as `ts_all` are needed when importing into
dynamodb as they are materialized and stored into dynamo.

#### Modification

re-adds virtual tileset creation before applying the diff. 

#### Checklist

_If not applicable, provide explanation of why._

- [ ] Tests updated
- [ ] Docs updated
- [ ] Issue linked in Title
  • Loading branch information
blacha authored Dec 6, 2023
1 parent c9099cf commit 8bf9be9
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions packages/cli/src/cli/config/action.import.ts
Original file line number Diff line number Diff line change
Expand Up @@ -77,6 +77,8 @@ export class CommandImport extends CommandLineAction {
logger.info({ config: this.target.value }, 'Import:Target:Load');
const configJson = await fsa.readJson<ConfigBundled>(this.target.value);
const mem = ConfigProviderMemory.fromJson(configJson);
mem.createVirtualTileSets();

setDefaultConfig(mem);
return mem;
}
Expand Down Expand Up @@ -107,6 +109,7 @@ export class CommandImport extends CommandLineAction {
logger.info({ config }, 'Import:Load');
const configJson = await fsa.readJson<ConfigBundled>(config);
const mem = ConfigProviderMemory.fromJson(configJson);
mem.createVirtualTileSets();

logger.info({ config }, 'Import:Start');
const objectTypes: Partial<Record<ConfigPrefix, number>> = {};
Expand Down

0 comments on commit 8bf9be9

Please sign in to comment.