Skip to content

feat: add AnalyticsDatasetDefinition to registry#1787

Open
aalhadxx wants to merge 1 commit into
forcedotcom:mainfrom
aalhadxx:feat/add-analyticsdatasetdefinition
Open

feat: add AnalyticsDatasetDefinition to registry#1787
aalhadxx wants to merge 1 commit into
forcedotcom:mainfrom
aalhadxx:feat/add-analyticsdatasetdefinition

Conversation

@aalhadxx

Copy link
Copy Markdown

What does this PR do?

Adds AnalyticsDatasetDefinition to the metadata registry for deploy/retrieve support.

What issues does this PR fix or reference?

Closes a gap identified in METADATA_SUPPORT.mdAnalyticsDatasetDefinition is listed as ❌ "Not supported, but support could be added."

Functionality Before

AnalyticsDatasetDefinition metadata type could not be deployed or retrieved using the CLI because it was absent from the registry.

Functionality After

AnalyticsDatasetDefinition is now recognized by the registry, enabling standard deploy/retrieve operations.

Metadata Registry: Successful Deploy and Retrieve (required if applicable)

Registry entry generated following the documented contribution guidelines at contributing/metadata.md.

Validation results:

  • yarn mocha test/registry/registryValidation.test.ts29,124 passing
  • Entry follows existing alphabetical ordering conventions in suffixes and types
  • No suffix collisions introduced

Adds AnalyticsDatasetDefinition to the metadata registry for deploy/retrieve support.

- Type supports deploy/retrieve per v67 metadata coverage report
- Generated using manual registry entry with validation test confirmation
- 29,124 registry validation tests passing

Refs: METADATA_SUPPORT.md gap
@aalhadxx

Copy link
Copy Markdown
Author

Local validation run on this branch:

$ yarn mocha test/registry/registryValidation.test.ts
29124 passing (838ms)
0 failing

The registry entry is clean and follows alphabetical ordering. Happy to address any feedback.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant