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

🐛 — Fetch all Gitlab groups of user #2413

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

Conversation

ldidry
Copy link

@ldidry ldidry commented Sep 12, 2024

Description 📣

The Gitlab integration only fetched the first page of user’s groups. This PR makes it fetch all the user’s groups.

Note: I choosed to fetch 10 groups per page as the project fetching code, which I just adapted, was using a pagination of 10 projects per page.

Type ✨

  • Bug fix
  • New feature
  • Breaking change
  • Documentation

Tests 🛠️

Steps to reproduce

  • have more than 20 groups (including subgroups) on your Gitlab account
  • create a Gitlab integration in Infisical
  • choose a group integration type
  • you don’t see all your groups

Test

I recompiled Infisical with my patch and it works, I can now see all my groups.


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

Successfully merging this pull request may close these issues.

1 participant