Suven-p

Results 22 comments of Suven-p

Has this been fixed? I couldn't replicate this.

@lohanidamodar This isn't complete but before I complete this could you check https://github.com/Suven-p/sdk-for-go/blob/7123c8348cb6360a0db1dfa6ba5831c5c8435579/databases/databases.go#L31 to see if it is a good idea. The usage for optional parameters is `database.List(databases.WithListQuery([queries]), databases.WithListSearch("Search term")`....

@lohanidamodar This should be done now. I am not sure how to test this since there are a lot of routes. Are the default tests sufficient? For the directory structure,...

The test case for last commit fails due to https://github.com/appwrite/appwrite/issues/5392 Should the end byte be set to the correct value (size -1)?

@eldadfux Can I work on this?

https://github.com/appwrite/sdk-generator/blob/55f8d82d90287a4f69b26297d559822e1d9475c7/templates/go/client.go.twig#L311 There doesn't seem to be a result field in current version of appwrite. Was it present in older versions of appwrite?

@phaus https://github.com/appwrite/sdk-generator/pull/647 is getting quite large. Should I split the PR?

This doesn't work with appwrite 1.3.1 because optional parameters do not accept falsy values. For example: when creating string attribute on a collection without a default value, older version allowed...

@phaus When creating an integer attribute on a collection, the user has to pass in argument for default since Go does not have optional arguments. For string arguments it can...