SetQueryParams supports map[string]interface{} maybe?

[walktall]

Something like this...

resty.R().
    SetQueryParams(map[string]interface{}{
        "name": "xxx",
        "age": 10,
    })

also
SetQueryParam(param string, value interface{})

[jeevatkm]

@zh-lab Actually URL query params are string always. That's why interface{} is not supported in those methods.

Do you have something else in your mind?

[walktall]

func (c *Client) SetQueryParams(params map[string]interface{}) *Client {
    for p, v := range params {
        c.QueryParam.Add(p, fmt.Sprint(v))
    }

    return c
}

func (c *Client) SetQueryParam(param string, value interface{}) *Client {
    c.QueryParam.Add(param, fmt.Sprint(value))
    return c
}

Then we could use int params without atoi?
Just my thoughts.

[jeevatkm]

Implementing this way is okay, however changing existing method signature might break user code. I will get back to you about method signature or if you have suggestion please let me know.

[walktall]

I see, thx!:smiley:

[jeevatkm]

Currently I'm thinking about unique method signature. If you have any name suggestion let me know. I will bring this interface{} support.

[voronovmaksim]

6 years have passed, but the api is not implemented(