Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Can this read child nodes as well for column values #6

Open
bheemreddy181-zz opened this issue May 18, 2020 · 5 comments
Open

Can this read child nodes as well for column values #6

bheemreddy181-zz opened this issue May 18, 2020 · 5 comments

Comments

@bheemreddy181-zz
Copy link

Is this limited to read data from a particular format ?
like mentioned in the docs

// the response data example
{
    "data": [
        {"time":1, "name": "test1"},
        {"time":2, "name": "test2"}
    ]
}

if i have multiple childs at different positions , can we pass the path for field it has to read ?

@bheemreddy181-zz
Copy link
Author

Like can i read totals.c and commit_id from this json

"totals": {
                                "C": 0,
                                "b": 0,
                                "d": 0,
                                "f": 28,
                                "h": 1255,
                                "M": 0,
                                "c": "41.76373",
                                "N": 0,
                                "p": 131,
                                "m": 1619,
                                "diff": null,
                                "s": 1,
                                "n": 3005
                            },
                            "commitid": "08aadebfacf6eaabc805ed2068d87ad613af558f",

@foora
Copy link
Owner

foora commented May 19, 2020

Now just can pass data like mentioned in the docs.
But I think it sounds good to read data from different positions by passing the path for field in configuration.
And also add an configuration to use user's own health check endpoint(#5 )
I will add them in a few days.

@bheemreddy181-zz
Copy link
Author

@foora when can I expect the above changes, I am kind of new to this language else could have created a PR for you soon

@bheemreddy181-zz
Copy link
Author

@foora just to circle back on this when do you think we can see new changes over here?

@bheemreddy181-zz
Copy link
Author

@foora Any update on this ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants