Skip to content

feat: Add coonfig-manager push services command#86

Open
dallinjsevy wants to merge 1 commit into
mainfrom
feature/config-manager-push-services
Open

feat: Add coonfig-manager push services command#86
dallinjsevy wants to merge 1 commit into
mainfrom
feature/config-manager-push-services

Conversation

@dallinjsevy
Copy link
Copy Markdown

No description provided.

@dallinjsevy dallinjsevy requested review from akynaston and phalestrivir and removed request for akynaston March 24, 2026 22:00
@dallinjsevy dallinjsevy force-pushed the feature/config-manager-push-services branch from 816bcf7 to ba61d3c Compare March 25, 2026 15:58
@phalestrivir phalestrivir force-pushed the feature/config-manager-push-services branch 2 times, most recently from d433546 to 1953746 Compare April 6, 2026 19:45
Copy link
Copy Markdown

@phalestrivir phalestrivir left a comment

Choose a reason for hiding this comment

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

I rebased, and also added some test configuration. One thing I noticed is that I don't think you are importing the next descendents correctly, so I added a few descendent configurations to the social provider service in the root realm that you can use for testing the descendents (the id-repositories service also has one too)

Comment thread src/configManagerOps/FrConfigServiceOps.ts Outdated
Comment thread src/configManagerOps/FrConfigServiceOps.ts
Comment thread test/e2e/config-manager-push-password-policy.e2e.test.js
Comment thread test/e2e/config-manager-push-services.e2e.test.js
@dallinjsevy dallinjsevy force-pushed the feature/config-manager-push-services branch 2 times, most recently from 1e011e2 to 36566cf Compare April 14, 2026 15:16
@dallinjsevy dallinjsevy force-pushed the feature/config-manager-push-services branch from 36566cf to f1341f1 Compare May 28, 2026 15:53
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