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

[INS-2259] [Improvement] validation of renaming the pipeline should be handled in the UI first. #609

Open
1 task done
iamnamananand996 opened this issue Oct 5, 2023 · 5 comments

Comments

@iamnamananand996
Copy link

iamnamananand996 commented Oct 5, 2023

Is There an Existing Issue for This?

  • I have searched the existing issues

Project

Instill Base

Is your Proposal Related to a Problem?

Currently the renaming the pipeline is directly calling the rename API to update the pipeline with having any validation in the UI.

CleanShot.2023-10-05.at.14.45.34.mp4

Describe Your Proposed Improvement

add the validation about the regex (^[a-z]([a-z0-9-]{0,61}[a-z0-9])?$) in the pipeline rename input field before call the rename API.

Highlight the Benefits

  • This is help to decrease overall number of network call

Anything Else?

No response

From SyncLinear.com | INS-2259

@iamnamananand996 iamnamananand996 changed the title [Improvement] validation of renaming the pipeline should be handled in the UI first. [INS-2259] [Improvement] validation of renaming the pipeline should be handled in the UI first. Oct 5, 2023
@github-actions
Copy link

github-actions bot commented Oct 5, 2023

This issue is a great way to kick-start your journey with our project, or to make a positive impact on open-source development. Jump in!

✨ Thank you for your contribution! ✨

@anurag629
Copy link

anurag629 commented Oct 5, 2023

Hello @iamnamananand996, I want to work on this. Can you assign this to me?

@EiffelFly
Copy link
Member

@anurag629 We are sorry that currently the console can't be contributed due to our local dev flow has some issues.

Will reach out to you once we clear this road block

@anurag629
Copy link

Will reach out to you once we clear this road block

Thanks for the information!

@Andrii-Vovk
Copy link

Hey! Leaving a comment here, just to get an update when something changes, I would like to make some contributions too.

@pinglin pinglin transferred this issue from instill-ai/community May 19, 2024
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

No branches or pull requests

4 participants