-
Notifications
You must be signed in to change notification settings - Fork 61
[Docs][Gitlab v2] Support Search Filtering on Groups, Projects, Issues and MRs #3067
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
[Docs][Gitlab v2] Support Search Filtering on Groups, Projects, Issues and MRs #3067
Conversation
…ts, Issues and MRs Signed-off-by: Dennis Bilson <dennis.bilson@port.io>
…grations-To-Narrow-Data-Fetching-for-Gitlab-v2
|
This pull request is automatically being deployed by Amplify Hosting (learn more). |
There was a problem hiding this 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".
| ```yaml | ||
| deleteDependentEntities: true | ||
| createMissingRelatedEntities: true | ||
| enableMergeEntity: true | ||
| # highlight-next-line |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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 👍 / 👎.
…grations-To-Narrow-Data-Fetching-for-Gitlab-v2
…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
|
Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits. |
|
🤖 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?
How to provide feedback We appreciate any thoughts or suggestions that help us improve. Your input helps us make our AI tools better. 🚀 Detected AI assistant: codex |
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>
…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
sivanel97
left a comment
There was a problem hiding this 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 :)
| :::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. | ||
| ::: |
There was a problem hiding this comment.
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.
docs/build-your-software-catalog/sync-data-to-catalog/git/gitlab-v2/examples.md
Outdated
Show resolved
Hide resolved
docs/build-your-software-catalog/sync-data-to-catalog/git/gitlab-v2/examples.md
Outdated
Show resolved
Hide resolved
| 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 |
There was a problem hiding this comment.
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.
docs/build-your-software-catalog/sync-data-to-catalog/git/gitlab-v2/examples.md
Outdated
Show resolved
Hide resolved
docs/build-your-software-catalog/sync-data-to-catalog/git/gitlab-v2/examples.md
Outdated
Show resolved
Hide resolved
…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
…iltering sections
shariff-6
left a comment
There was a problem hiding this 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
mk-armah
left a comment
There was a problem hiding this 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
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
/)/build-your-software-catalog/sync-data-to-catalog/git/gitlab-v2/examples?parameter=search#projects-and-issues)