Skip to content

Updates for Beacon#31

Open
ArkeBcacy wants to merge 85 commits into
Arke-Systems:mainfrom
ArkeBcacy:feature/merge-back-to-arke
Open

Updates for Beacon#31
ArkeBcacy wants to merge 85 commits into
Arke-Systems:mainfrom
ArkeBcacy:feature/merge-back-to-arke

Conversation

@ArkeBcacy

Copy link
Copy Markdown

Added ability to serialize to json along with yaml (default) option.
Added language support for serialization. Non-default language serialization includes locale in the file name, ex SomeFileName.zh-cn.yaml.
Added support for including/excluding specific asset serialization to Contentstack.
Added serialization of Labels.
Corrected issue with asset duplication when Contentstack API times out during pushes.
Updated clear command to support removing only specific content types.
Added support for beacon image replacement in RTE fields.
Added support to include/exclude certain content types for serialization.

This comment was marked as resolved.

Copilot AI left a comment

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.

Pull request overview

Copilot reviewed 130 out of 131 changed files in this pull request and generated 4 comments.

Comment thread .github/workflows/publish-package.yml
Comment thread cli/src/cs/labels/import.ts Outdated
Comment thread cli/src/cs/labels/export.ts Outdated
Comment thread cli/src/cs/api/retryWithBackoff.test.ts Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Bill Cacy <98779849+ArkeBcacy@users.noreply.github.com>

Copilot AI left a comment

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.

Pull request overview

Copilot reviewed 130 out of 131 changed files in this pull request and generated 4 comments.

Comment on lines +3 to +6
describe('shouldSkipFallbackLocale behavior', () => {
// Note: shouldSkipFallbackLocale is a private function in toFilesystem.ts
// These tests document the expected behavior after the fix
it('should detect fallback for base locale files (useLocaleSuffix=false)', () => {
Comment on lines +4 to +7
describe('getMasterLocale behavior', () => {
// Note: getMasterLocale is a private function in toContentstack.ts
// These tests document the expected behavior

Comment on lines +70 to +75
async function deleteIfExists(client: Client, uid: string) {
try {
await deleteTaxonomy(client, uid);
} catch {
// Taxonomy doesn't exist, which is fine
}
Comment thread cli/package.json Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Bill Cacy <98779849+ArkeBcacy@users.noreply.github.com>
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