Skip to content

feat: filter out the GitLab repos that users only have the Guest perm…#5664

Merged
mindlesscloud merged 2 commits into
apache:mainfrom
merico-ai:feat#5104
Jul 25, 2023
Merged

feat: filter out the GitLab repos that users only have the Guest perm…#5664
mindlesscloud merged 2 commits into
apache:mainfrom
merico-ai:feat#5104

Conversation

@abeizn
Copy link
Copy Markdown
Contributor

@abeizn abeizn commented Jul 12, 2023

Summary

don't merge after v0.18.0-beta1

Filter out the GitLab repos that users only have the Guest permission and archived projects

Does this close any open issues?

Closes #5104

Screenshots

28e2213d-f89f-4e52-8317-2f5a708189c8

img_v2_c2a72dd2-8581-4a91-a1e1-6b596ffab9eg

Other Information

Any other information that is important to this PR.

@abeizn abeizn marked this pull request as ready for review July 12, 2023 13:08
}
} else {
query.Set("with_shared", "false")
if gid[:6] == "group:" {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Using strings.HasPrefix may be a better solution

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Other remote APIs also have the same problem, or should it be fixed in the subsequent PR?

Copy link
Copy Markdown
Contributor

@mindlesscloud mindlesscloud left a comment

Choose a reason for hiding this comment

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

lgtm

@mindlesscloud mindlesscloud merged commit 2849e45 into apache:main Jul 25, 2023
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.

[Feature][GitLab] Filter out the GitLab repos that users only have the Guest permission when setting data scope

2 participants