Typescript typings wrong for copyObject operation

[phal0r]

Given the latest releases:

"minio": "^7.0.32",
"@types/minio": "^7.0.15"

Typings are wrong for copyObject. The overloads for optional CopyConditions are missing. Passing null also leads to the error, although optional CopyConditions or passing null is allowed and supported in the source.

[6XGate]

There are many issues with the typings. Another is the BucketItem type is defined such that it would seem all members are present, which isn't the case. Prefix-only entries only have prefix and size where as object entries has name, lastModified, etag, and size.

It also requires the access and secret key for the client, which are actually optional.

I'm sure there are other issues.

[aldy505]

@phal0r Since we've migrated @types/minio to this repo, can you please take a look at:

minio-js/src/minio.d.ts

Lines 404 to 416 in 23ad6e2

	copyObject(
	bucketName: string,
	objectName: string,
	sourceObject: string,
	conditions: CopyConditions,
	callback: ResultCallback<BucketItemCopy>,
	): void
	copyObject(
	bucketName: string,
	objectName: string,
	sourceObject: string,
	conditions: CopyConditions,
	): Promise<BucketItemCopy>

and confirm whether that is correct?

p.s. me and @trim21 are not minio employee

[trim21]

@phal0r Since we've migrated @types/minio to this repo, can you please take a look at:

minio-js/src/minio.d.ts

Lines 404 to 416 in 23ad6e2

	copyObject(
	bucketName: string,
	objectName: string,
	sourceObject: string,
	conditions: CopyConditions,
	callback: ResultCallback<BucketItemCopy>,
	): void
	copyObject(
	bucketName: string,
	objectName: string,
	sourceObject: string,
	conditions: CopyConditions,
	): Promise<BucketItemCopy>

and confirm whether that is correct?

p.s. me and @trim21 are not minio employee

it's still wrong ( or maybe "incomplete" ), I didn't fixed this when importing minio.d.ts.

missing overloads:

 copyObject( 
   bucketName: string, 
   objectName: string, 
   sourceObject: string, 
   callback: ResultCallback<BucketItemCopy>, 
 ): void 

 copyObject( 
   bucketName: string, 
   objectName: string, 
   sourceObject: string, 
   conditions?: CopyConditions, 
 ): Promise<BucketItemCopy> 

（we realy should consider dropping callback API)