Skip to content
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: duplicate repo entry frontend validation #580

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ezhil56x
Copy link
Contributor

@ezhil56x ezhil56x commented May 17, 2024

Duplicate repo entry frontend validation

Description

Adding the same repo twice in a multi project workspace will do frontend validation
Closes #550
/claim #550

  • This change requires a documentation update
  • I have made corresponding changes to the documentation

Related Issue(s)

This PR addresses issue #550

Screenshots

2024-05-18.01-04-16.mp4

@ezhil56x
Copy link
Contributor Author

@Tpuljak
Once #578 is merged. I'll rebase my code

Signed-off-by: Ezhil Shanmugham <ezhil930@gmail.com>
Copy link
Member

@Tpuljak Tpuljak left a comment

Choose a reason for hiding this comment

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

@ezhil56x thanks for the PR!

The user should be let know there are duplicate repo entries before having submitted the form
This quote from the issue was hinting more towards not breaking the command if the UI detects a duplicate repo entry.
We should just notify the user via an error message in the UI that we detected a duplicate entry and that the user should provide a different repo.

Note: This should be achieved in both --manual mode and TUI mode. --manual mode is that you recorded, it's also automatically triggered if the user does not have a git provider set. I suggest you add a git provider with daytona git-provider add and then test both daytona create --multi-project and daytona create --multi-project --manual.

I'll convert this PR to draft. When you're done, mark it as ready for review and we'll take a look again. Thanks!

@Tpuljak Tpuljak marked this pull request as draft May 18, 2024 09:31
@daytonaBot
Copy link
Collaborator

This Pull Request has been automatically marked as "stale" because it has not had recent activity (for 14 days). It will be closed if no further activity occurs. Thank you for your contribution.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Duplicate repo entry frontend validation
3 participants