Skip to content

feat: add layer deletion schemas, types and constants (MAPCO-7285)#210

Merged
almog8k merged 5 commits into
alphafrom
feat/delete-layer-MAPCO-7285
Jun 29, 2026
Merged

feat: add layer deletion schemas, types and constants (MAPCO-7285)#210
almog8k merged 5 commits into
alphafrom
feat/delete-layer-MAPCO-7285

Conversation

@almog8k

@almog8k almog8k commented Jun 25, 2026

Copy link
Copy Markdown
Collaborator
Question Answer
Bug fix
New feature
Breaking change
Deprecations
Documentation
Tests added
Chore

Related issues: MAPCO-7285

Further information:

Adds the shared deletion module for the layer deletion feature, exported from the constants, schemas and types barrels.

  • ConstantsDeletionJobTypes (Delete_Layer) and DeletionTaskTypes (delete, full-layer-tiles-deletion, artifacts-deletion).
  • SchemasdeleteLayerJobParamsSchema, deleteTaskParamsSchema, layerTilesDeletionParamsSchema, artifactsDeletionParamsSchema (and the shared sourceProviderSchema over the existing core SourceType).
  • TypesDeleteLayerJobParams, DeleteTaskParams, LayerTilesDeletionParams, ArtifactsDeletionParams inferred from the schemas.

Base branch: targets alpha (the 8.3.0-alpha.x prerelease line for this epic), not master. The published latest version stays at 8.2.0 while the epic is in progress.

Comment thread src/constants/deletion/constants.ts Outdated

export const DeletionTaskTypes = {
Delete: 'delete',
LayerTilesDeletion: 'full-layer-tiles-deletion',

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

keep as 'tiles-deletion' as the job type itself is "Delete-Layer"

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changed to tiles-deletion

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changed

Comment thread src/schemas/deletion/task.schema.ts Outdated

export const artifactsDeletionParamsSchema = z
.object({
sourceProvider: sourceProviderSchema,

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

add optional "bucket" param

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added

@almog8k almog8k merged commit 8d50c52 into alpha Jun 29, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants