Skip to content
This repository has been archived by the owner on Jan 15, 2024. It is now read-only.

Fixed folder pagination #134

Merged

Conversation

Jramirezg
Copy link
Contributor

@Jramirezg Jramirezg commented Feb 6, 2023

This MR offers support to the terraform provider for Grafana for folder search. Currently a folder search does not include results beyond 1000 items since the pagination was not in place.

@CLAassistant
Copy link

CLAassistant commented Feb 6, 2023

CLA assistant check
All committers have signed the CLA.

@davidnewhall
Copy link
Contributor

This looks familiar. :D <3

@Jramirezg
Copy link
Contributor Author

@davidnewhall This is needed for an internal project at Booking.com. Any chance you could approve it, please? :)

@davidnewhall
Copy link
Contributor

@Jramirezg I absolutely can, but I don't have any permissions on this repo.

@Jramirezg
Copy link
Contributor Author

Jramirezg commented Mar 27, 2023

Any idea of who I can approach that can merge it?

@davidnewhall
Copy link
Contributor

davidnewhall commented Mar 27, 2023

One of the Grafana devs came through and merged mine without any 'asking'. I suspect they look from time to time. Your contribution looks a lot like mine, so I imagine they won't have a problem accepting it. If you have enterprise support, maybe ask your TAM.

Copy link
Member

@julienduchesne julienduchesne left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@julienduchesne julienduchesne merged commit 303710b into grafana:master Mar 29, 2023
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.

4 participants