You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
fix: correct asset_fields[] parameter names for DAM 2.0
The previous PR #336 partially fixed asset_fields parameters but used
incorrect field names. This commit corrects all parameter names to match
the CDA API specification.
Changes:
- Changed 'ai_suggested' to 'ai_generated_metadata' in all test files
- Updated SDK documentation to reflect correct parameter names
- All asset_fields tests now pass (verified locally)
Valid asset_fields[] values per API:
- user_defined_fields
- embedded_metadata (not 'embedded')
- ai_generated_metadata (not 'ai_suggested')
- visual_markups
API Error that revealed the issue:
"Invalid values found, supported values are user_defined_fields,
embedded_metadata, ai_generated_metadata, visual_markups"
Files Modified:
- test/api/asset.spec.ts
- test/api/asset-query.spec.ts
- test/api/entry.spec.ts
- test/api/entries.spec.ts
- src/assets/asset.ts (JSDoc)
- src/query/asset-query.ts (JSDoc)
- src/entries/entry.ts (JSDoc)
- src/entries/entries.ts (JSDoc)
Test Results: All 16 asset_fields tests passing (8 API + 8 unit)
0 commit comments