community icon indicating copy to clipboard operation
community copied to clipboard

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

Open iamnamananand996 opened this issue 1 year ago • 5 comments

Is There an Existing Issue for This?

  • [X] 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.

https://github.com/instill-ai/community/assets/31537362/2d1ea9d5-398d-45ef-90a9-39e5e7f974a5

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 avatar Oct 05 '23 09:10 iamnamananand996

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!

:sparkles: Thank you for your contribution! :sparkles:

github-actions[bot] avatar Oct 05 '23 09:10 github-actions[bot]

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

anurag629 avatar Oct 05 '23 09:10 anurag629

@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

EiffelFly avatar Oct 05 '23 12:10 EiffelFly

Will reach out to you once we clear this road block

Thanks for the information!

anurag629 avatar Oct 05 '23 13:10 anurag629

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

Andrii-Vovk avatar Jan 18 '24 09:01 Andrii-Vovk