Convert Routes to use Constants

[saivineeth100]

Existing

private readonly Dictionary<string, string> _routes = new Dictionary<string, string>
{
    ["parameters"] = "/parameters",

}

Suggested Add to Constants.cs or Add new File Routes.cs

    public static class Routes
    {
        public const string Parameters = "/parameters";
        public static class Auth
        {
            public const string Token = "/session/token";
            public const string RefreshToken = "/session/refresh_token";
        }

    }

When using we can use

Post(Routes.Auth.Token, reqParams)

[ajinasokan]

Thanks for the suggestion. But this will be quite a significant change. Will consider this for the next major version. I will keep the issue open.

[saivineeth100]

Thanks for the suggestion. But this will be quite a significant change. Will consider this for the next major version. I will keep the issue open.

Can I contribute to this or any suggestions via pull requests?

[ajinasokan]

I actually have the patch ready. But just not merged yet. Since this is changing code related to all APIs, it will have to be tested as a whole and that is time consuming. That is why I'm delaying it till a major version or a refactor.

[saivineeth100]

I am asking in general, can outsiders contribute to Zerodha repos?

[ajinasokan]

Yeah sure. Just keep in mind, if it is a large patch then make sure to create an issue with your suggestion and seek maintainers opinion.