-
Notifications
You must be signed in to change notification settings - Fork 6
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.
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
feat: setup a synchronization plan if the resources are in error #248
Conversation
Pull Request Report PR Title ✅ Title follows the conventional commit spec. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code looks good!
I tagged @jpmarceau 'cause I ain't sure about the message tho
Co-authored-by: Louis Bompart <[email protected]>
Co-authored-by: Louis Bompart <[email protected]>
Co-authored-by: Louis Bompart <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💯 !
Proposed changes
If the snapshot and the target organization are not in sync, the command will print an URL to the Admin Console so the user can synchronize his organization.
The CLI is not handling any synchronization logic at the moment.
Testing
CDX-370