-
Notifications
You must be signed in to change notification settings - Fork 219
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
fix: multi-project create not working #578
Conversation
41c0430
to
4159a1b
Compare
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.
Nice catch!
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.
Approved before, but I found an issue.
When you use daytona create --multi-project
while having a git provider, the second repo (and all after that) are ignored. That's also what the linter is throwing.
The if-else statement was there to ensure this. If the user does not provide the repo through the TUI (the user selects the manual input) then you need to show them the manual url input form, otherwise, just the "Add more" switch.
My screenshot was taken after I selected a repo from the TUI.
Signed-off-by: tarunrajput <tarunrajput1337@gmail.com>
4159a1b
to
6cdad61
Compare
@Tpuljak, it should be fixed now |
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.
Nice job! My mistake in the first place 😄
Description
fix
--multi-project
create not workingRelated Issue(s)
Closes #577
/claim #577