Skip to content

Conversation

@dennis-bilson-port
Copy link
Member

This PR adds support for optional Gitlab selectors when fetching the groups, project, issue and merge request kinds, enabling precise scoping and reduced payload.

Updated docs pages

Please also include the path for the updated docs

  • Quickstart (/)
  • Blueprint (/build-your-software-catalog/sync-data-to-catalog/git/gitlab-v2/examples?parameter=search#projects-and-issues)

…ts, Issues and MRs

Signed-off-by: Dennis Bilson <dennis.bilson@port.io>
…grations-To-Narrow-Data-Fetching-for-Gitlab-v2
@dennis-bilson-port dennis-bilson-port marked this pull request as ready for review November 23, 2025 13:19
@aws-amplify-eu-west-1
Copy link

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-3067.d2ngvl90zqbob8.amplifyapp.com

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment on lines 69 to 73
```yaml
deleteDependentEntities: true
createMissingRelatedEntities: true
enableMergeEntity: true
# highlight-next-line

Choose a reason for hiding this comment

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

P1 Badge Add showLineNumbers to new YAML snippets

The root AGENTS.md requires every multi-line code block to include showLineNumbers, but the newly added YAML example here lacks that attribute. The same omission appears in the new project, merge-request, and issue search examples later in the file, so these snippets currently violate the mandatory code-example formatting guideline.

Useful? React with 👍 / 👎.

@dennis-bilson-port dennis-bilson-port marked this pull request as draft December 1, 2025 12:54
…grations-To-Narrow-Data-Fetching-for-Gitlab-v2
@sivanel97 sivanel97 added the WIP label Dec 1, 2025
…grations-To-Narrow-Data-Fetching-for-Gitlab-v2
…tabbed examples for groups, projects, issues, merge requests, members, pipelines, jobs, releases, and tags.

Signed-off-by: Dennis Bilson <dennis.bilson@port.io>
…grations-To-Narrow-Data-Fetching-for-Gitlab-v2
@dennis-bilson-port dennis-bilson-port marked this pull request as ready for review December 2, 2025 17:33
@chatgpt-codex-connector
Copy link

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Credits must be used to enable repository wide code reviews.

@github-actions
Copy link
Contributor

github-actions bot commented Dec 2, 2025

🤖 AI Assistant Feedback Request

I noticed that codex authored a comment!

To help us improve our AI assistance, could you please let us know:

Did the AI assistant help you?

  • Helpful - The AI provided useful assistance
  • Not helpful - The AI didn't provide useful assistance
  • 🔄 Partially helpful - The AI was somewhat useful but could be better

How to provide feedback
Please reply to this comment with /feedback/ followed by any feedback you'd like to share.
For example: /feedback The AI's answer was unclear. or /feedback Great job! Very helpful.

We appreciate any thoughts or suggestions that help us improve.

Your input helps us make our AI tools better. 🚀

Detected AI assistant: codex

dennis-bilson-port and others added 3 commits December 2, 2025 18:10
Signed-off-by: Dennis Bilson <dennis.bilson@port.io>
…grations-To-Narrow-Data-Fetching-for-Gitlab-v2
…includeActiveProjects` to `includeOnlyActiveProjects` in GitLab v2 examples.

Signed-off-by: Dennis Bilson <dennis@teksol.com.gh>
@dennis-bilson-port dennis-bilson-port self-assigned this Dec 10, 2025
dennis-bilson-port and others added 6 commits December 10, 2025 13:43
…grations-To-Narrow-Data-Fetching-for-Gitlab-v2
…grations-To-Narrow-Data-Fetching-for-Gitlab-v2
…ct search filtering effects

Signed-off-by: Dennis Bilson <dennis.bilson@port.io>
…grations-To-Narrow-Data-Fetching-for-Gitlab-v2
…grations-To-Narrow-Data-Fetching-for-Gitlab-v2
…grations-To-Narrow-Data-Fetching-for-Gitlab-v2
Copy link
Contributor

@sivanel97 sivanel97 left a comment

Choose a reason for hiding this comment

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

Left some small comments :)

Comment on lines 220 to 222
:::info Important
The `updatedAfter` parameter only affects issues that are not in the "opened" state. Open issues will always be synced regardless of their last update time.
:::
Copy link
Contributor

Choose a reason for hiding this comment

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

Please include a title admonition, this makes the admonition's purpose clearer and increases the chance that readers will not skip it if it's relevant to them.

The `labels` selector allows you to filter issues based on a comma-separated list of labels.
By default, if not specified, all issues for the authorized groups will be synced.

:::info Important
Copy link
Contributor

Choose a reason for hiding this comment

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

Always include a title for admonitions, this makes the admonition's purpose clearer and increases the chance that readers will not skip it if it's relevant to them.

dennis-bilson-port and others added 5 commits December 22, 2025 12:11
…ab-v2/examples.md

Co-authored-by: sivanel97 <sivane@getport.io>
…ab-v2/examples.md

Co-authored-by: sivanel97 <sivane@getport.io>
…ab-v2/examples.md

Co-authored-by: sivanel97 <sivane@getport.io>
…ab-v2/examples.md

Co-authored-by: sivanel97 <sivane@getport.io>
…grations-To-Narrow-Data-Fetching-for-Gitlab-v2
Copy link
Contributor

@shariff-6 shariff-6 left a comment

Choose a reason for hiding this comment

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

LGTM

…grations-To-Narrow-Data-Fetching-for-Gitlab-v2
Copy link
Member

@mk-armah mk-armah left a comment

Choose a reason for hiding this comment

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

LGTM

…grations-To-Narrow-Data-Fetching-for-Gitlab-v2
…grations-To-Narrow-Data-Fetching-for-Gitlab-v2
@hadar-co hadar-co merged commit 89eda37 into main Dec 23, 2025
5 checks passed
@hadar-co hadar-co deleted the PORT-16838-Docs-Add-Selector-Filters-To-Integrations-To-Narrow-Data-Fetching-for-Gitlab-v2 branch December 23, 2025 13:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants