Commit5b54276

committed

[Fix]: query deployment param

1 parentb35861b commit5b54276Copy full SHA for 5b54276

File tree

+12

-13

lines changed

+12

-13

lines changed

Lines changed: 7 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -17,6 +17,12 @@ export const queryConfig: DeployableItemConfig = {`
`17`	`17`	`label:'Update Dependencies If Needed',`
`18`	`18`	`type:'checkbox',`
`19`	`19`	`defaultValue:false`
	`20`	`+},`
	`21`	`+{`
	`22`	`+name:'deployCredential',`
	`23`	`+label:'Overwrite Credentials',`
	`24`	`+type:'checkbox',`
	`25`	`+defaultValue:false`
`20`	`26`	`}`
`21`	`27`	`],`
`22`	`28`	`prepareParams:(item:Query,values:any,sourceEnv:Environment,targetEnv:Environment)=>{`
`@@ -26,6 +32,7 @@ export const queryConfig: DeployableItemConfig = {`
`26`	`32`	`queryId:item.id,`
`27`	`33`	`updateDependenciesIfNeeded:values.updateDependenciesIfNeeded,`
`28`	`34`	`queryGid:item.gid,`
	`35`	`+deployCredential:values.deployCredential??false`
`29`	`36`	`};`
`30`	`37`	`},`
`31`	`38`	`execute:(params:any)=>deployQuery(params)`

Lines changed: 2 additions & 9 deletions

Original file line number	Diff line number	Diff line change
`@@ -5,20 +5,13 @@ import { Environment } from '../types/environment.types';`
`5`	`5`	`import{deployWorkspace}from'../services/workspace.service';`
`6`	`6`	`import{Workspace}from'../types/workspace.types';`
`7`	`7`
`8`		`-`
`9`		`-`
`10`	`8`	`exportconstworkspaceConfig:DeployableItemConfig={`
`11`	`9`
`12`	`10`	`// Deploy configuration`
`13`	`11`	`deploy:{`
`14`	`12`	`singularLabel:'Workspace',`
`15`	`13`	`fields:[`
`16`		`-{`
`17`		`-name:'deployCredential',`
`18`		`-label:'Overwrite Credentials',`
`19`		`-type:'checkbox',`
`20`		`-defaultValue:false`
`21`		`-}`
	`14`	`+// Removed deployCredential field as workspaces don't need credential overwrite`
`22`	`15`	`],`
`23`	`16`	`prepareParams:(item:Workspace,values:any,sourceEnv:Environment,targetEnv:Environment)=>{`
`24`	`17`	`if(!item.gid){`
`@@ -30,7 +23,7 @@ export const workspaceConfig: DeployableItemConfig = {`
`30`	`23`	`envId:sourceEnv.environmentId,`
`31`	`24`	`targetEnvId:targetEnv.environmentId,`
`32`	`25`	`workspaceId:item.gid,`
`33`		`-deployCredential:values.deployCredential??false`
	`26`	`+// RemoveddeployCredentialparameter`
`34`	`27`	`};`
`35`	`28`	`},`
`36`	`29`	`execute:(params:any)=>deployWorkspace(params)`

Lines changed: 3 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -16,6 +16,7 @@ export interface MergedQueriesResult {`
`16`	`16`	`queryId:string;`
`17`	`17`	`updateDependenciesIfNeeded?:boolean;`
`18`	`18`	`queryGid:string;`
	`19`	`+deployCredential:boolean;`
`19`	`20`	`}`
`20`	`21`
`21`	`22`
`@@ -71,7 +72,8 @@ export interface MergedQueriesResult {`
`71`	`72`	`envId:params.envId,`
`72`	`73`	`targetEnvId:params.targetEnvId,`
`73`	`74`	`queryId:params.queryId,`
`74`		`-updateDependenciesIfNeeded:params.updateDependenciesIfNeeded??false`
	`75`	`+updateDependenciesIfNeeded:params.updateDependenciesIfNeeded??false,`
	`76`	`+deployCredential:params.deployCredential`
`75`	`77`	`}`
`76`	`78`	`});`
`77`	`79`	`if(response.status===200){`

Lines changed: 0 additions & 3 deletions

Original file line number	Diff line number	Diff line change
`@@ -84,7 +84,6 @@ export async function deployWorkspace(params: {`
`84`	`84`	`envId:string;`
`85`	`85`	`targetEnvId:string;`
`86`	`86`	`workspaceId:string;`
`87`		`-deployCredential:boolean;// Mandatory parameter`
`88`	`87`	`}):Promise<boolean>{`
`89`	`88`	`try{`
`90`	`89`	`// Use the new endpoint format with only essential parameters`
`@@ -93,7 +92,6 @@ export async function deployWorkspace(params: {`
`93`	`92`	`orgGid:params.workspaceId,// Using workspaceId as orgGid`
`94`	`93`	`envId:params.envId,`
`95`	`94`	`targetEnvId:params.targetEnvId,`
`96`		`-deployCredential:params.deployCredential`
`97`	`95`	`}`
`98`	`96`	`});`
`99`	`97`
`@@ -107,7 +105,6 @@ export async function deployWorkspace(params: {`
`107`	`105`	`);`
`108`	`106`	`}`
`109`	`107`
`110`		`-`
`111`	`108`	`returnresponse.status===200;`
`112`	`109`	`}catch(error){`
`113`	`110`	`consterrorMessage=errorinstanceofError ?error.message :'Failed to deploy workspace';`

Comments

(0)