-
Notifications
You must be signed in to change notification settings - Fork 0
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
Should be able to import personal projects #7
Comments
Idk, you're the author :P You tell me |
Unsure what is meant by personal projects here, and if it is retrieved differently from "normal" projects or not. Probably should have added that in my previous comment 🦘 |
@Alexamakans IIRC then this is only an issue when importing projects in Wharf from GitLab with the group specified but not the project name. For example:
Probably because if not supplying a project name when importing via Wharf, it performs a group project listing, when it instead should be a little smarter and do a user project listing if the specified Wharf group ends up being a GitLab user. |
That part of the issue description I cannot vouch for. I am not the original author of this issue, as it was a work order that was copied over from our previously used issue tracker/ITSM ("The Cure"). I do not see any need for any frontend changes. Rather that the wharf-provider-gitlab deals with the two alternative Wharf group type abstraction instead of like adding a |
Currently GitLab importer are able to import projects from selected group. When we do not indicate the group it will take all projects that user has access to including personal projects. On the list it is shown as 'username/projectname'. If someone will decide to import only personal projects an error will occur. Things to do:
add get list personal project functionality to git lab fetcher
on import group add aditional check if delivered group name is a user name. If it is a group or a username, it should be supplied as a separate input.
write UT to new functionality
Changes in frontend should be included in this task
The text was updated successfully, but these errors were encountered: