Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

fix a small bug #87

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

fix a small bug #87

wants to merge 1 commit into from

Conversation

changdingfang
Copy link

I encountered a situation where the value of "totalCount" was inconsistent with the total length of "team", which resulted in a loop that could not be returned.
So I modified the break condition and solved the problem.

Sorry, my English is not good, this is done with Google Translate, I hope my expression and translation are very clear

@amotl
Copy link

amotl commented Feb 6, 2022

Dear @changdingfang,

we recently forked this repository to 1, see #88. Over there, we will be very happy to accept your patch.

With kind regards,
Andreas.

Footnotes

  1. https://github.com/panodata/grafana-client

amotl pushed a commit to grafana-toolbox/grafana-client that referenced this pull request Jun 23, 2022
This improves a situation where the value of "totalCount" was
inconsistent with the total length of "teams", which resulted in a loop
that could not be returned.

References:
- m0nhawk/grafana_api#87
- #12
@amotl
Copy link

amotl commented Jun 23, 2022

Dear @changdingfang,

we have been tracking this topic with grafana-toolbox/grafana-client#12, and we just included your fix with grafana-toolbox/grafana-client@3d6d9c985.

Thank you again and with kind regards,
Andreas.

amotl pushed a commit to grafana-toolbox/grafana-client that referenced this pull request Jun 25, 2022
This improves a situation where the value of "totalCount" was
inconsistent with the total length of "teams", which resulted in a loop
that could not be returned.

References:
- m0nhawk/grafana_api#87
- #12
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants