Skip to content

feat: content item command improvements #239

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
Benj0s merged 62 commits into from
Nov 14, 2025
Merged

feat: content item command improvements #239

Benj0s merged 62 commits into from
Nov 14, 2025

Conversation

@Benj0s
Copy link
Contributor

@Benj0s Benj0s commented Oct 28, 2025

No description provided.

Benj0s and others added 30 commits August 19, 2025 12:57
@Benj0s
test: update to latest version of jest
e547412
@Benj0s
feat: adding burstable queue
703f579
@Benj0s
feat: adding publishing and publishing job services - wip
4c72bb4
@deeebeee3
feat: first pass content item publish (not tested) wip
92d98e4
@deeebeee3
refactor: update options, command options, modify tests
228f4d9
@deeebeee3
feat: content item import publish to use new services
070d854
@deeebeee3
refactor: update return types and publishing service actions - wip
19084d9
@Benj0s
feat: pass headers via custom dc http client to match sdk
a4728ca
@Benj0s
test: updating import tests to use new publishing service
8ca20d1
@Benj0s
feat: removing redundant publishing flags
29921b3
@deeebeee3
test: updating publish tests to use new publishing service
49a1fa2
@Benj0s
feat: add publish progress and only log to log file
c5bc0bf
@Benj0s
feat: allow publish checks to occur without user input
5c768d9
@deeebeee3
test: adding tests for publishing services
e0c13b7
@Benj0s
fix: improve logging when publishing content
b637988
@Benj0s
fix: moving publish complete log to correct scope
2f0caba
@Benj0s
feat: add min time to burstable queue to spread load
3286ecd
@Benj0s
fix: lowering min time to make sure we use the full burst interval re…
0b893cb 
…servoir
@deeebeee3
feat: add unpublish command
056743b
@Benj0s
Merge branch 'master' into feature/publish-queue-improvements
4b47d06
@deeebeee3
Merge branch 'feature/publish-queue-improvements' into feature/unpublish
d76d481
@deeebeee3
fix: ensure action gets called
149b812
@deeebeee3
refactor: unpublish method, add tests to cover all statuses
d5c6104
@deeebeee3
fix: prevent unpublish for item with status NONE
98ccb2d
@deeebeee3
fix: remove unpublish children logic
b9a17b9
@deeebeee3
fix: log immediate children
28f5311
@Benj0s
fix: reintroduce publish content tree child item filtering
4fe256a
@Benj0s
Merge branch 'feature/publish-queue-improvements' into feature/unpublish
d3b5c3a
@Benj0s
feat: adding commands for content sync jobs - wip
11cb298
@Benj0s
feat: making content item fetch reusable
1491b6e
@github-actions github-actions bot added the test label Oct 28, 2025
@amplience-snyk
Copy link

amplience-snyk commented Oct 28, 2025 •
edited
Loading

✅ Snyk checks have passed. No issues have been found so far.

Status Scanner Critical High Medium Low Total (0)
✅ Open Source Security 0 0 0 0 0 issues
✅ Licenses 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@Benj0s Benj0s force-pushed the feature/content-item-command-improvements branch from dab35a1 to c1ccaa2 Compare November 3, 2025 09:05
@Benj0s Benj0s marked this pull request as ready for review November 5, 2025 16:42
scottstore4
scottstore4 approved these changes Nov 10, 2025
View reviewed changes
Copy link

@scottstore4 scottstore4 left a comment

Choose a reason for hiding this comment

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

Tested and approved

@Benj0s Benj0s changed the base branch from feature/content-sync to master November 10, 2025 15:27
@Benj0s Benj0s merged commit 5d02a0b into master Nov 14, 2025
9 checks passed
@amp-automation amp-automation mentioned this pull request Nov 14, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@deeebeee3 deeebeee3 deeebeee3 approved these changes

@neilmistryamplience neilmistryamplience neilmistryamplience approved these changes

@scottstore4 scottstore4 scottstore4 approved these changes

@tgallimore tgallimore Awaiting requested review from tgallimore

@thinton26 thinton26 Awaiting requested review from thinton26

Assignees

No one assigned

Labels

test

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@Benj0s @amplience-snyk @deeebeee3 @neilmistryamplience @scottstore4