Skip to content

136805 - Update Customer Environment Deployment Parameters in a specific request#113

Open
ritanorinho wants to merge 1 commit intomainfrom
main-136805-updateParametersInASpecificRequest
Open

136805 - Update Customer Environment Deployment Parameters in a specific request#113
ritanorinho wants to merge 1 commit intomainfrom
main-136805-updateParametersInASpecificRequest

Conversation

@ritanorinho
Copy link
Contributor

Changes

  • Updated logic so that customer environment deployment parameters are now updated through a specific request instead of the Generic Update Customer Environment endpoint.
  • Retrieval of termination logs has been moved to the specific request, since they are no longer loaded directly with the Customer Environment.

@github-actions
Copy link
Contributor

Code Coverage

Package Line Rate Branch Rate Health
Cmf.CustomerPortal.Sdk.Common 18% 17%
Summary 18% (257 / 1454) 17% (85 / 506)

var output = await (new GetCustomerEnvironmentByIdInput() { CustomerEnvironmentId = ceId }.GetCustomerEnvironmentByIdAsync(true));
CustomerEnvironment ce = output.CustomerEnvironment;
Session.LogError($"\nCustomer Environment {ce.Id} did not terminate sucessully. Termination logs:\n {ce.TerminationLogs}\n");
var output = await (new GetCustomerEnvironmentTerminationLogsInput() { CustomerEnvironmentId = ceId }.GetCustomerEnvironmentTerminationLogsAsync(true));
Copy link
Collaborator

Choose a reason for hiding this comment

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

I'm going to do a PR with just this change since it is related with a different feature. Don't forget to re-bump and merge afterwards.

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.

3 participants