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

[NV-3072] Workflow update button should show loading state when clicked #4703

Closed
jainpawan21 opened this issue Oct 30, 2023 · 8 comments · Fixed by #4781
Closed

[NV-3072] Workflow update button should show loading state when clicked #4703

jainpawan21 opened this issue Oct 30, 2023 · 8 comments · Fixed by #4781
Assignees

Comments

@jainpawan21
Copy link
Member

jainpawan21 commented Oct 30, 2023

What?

Whenever we add new step or change anything in any of the workflow step, update button becomes active

Why? (Context)

After updating the step or workflow settings, when user clicks on the update button. it gets freezes for few seconds. User may click multiple times.

Screen.Recording.2023-10-24.at.11.20.49.PM.mov

Definition of Done

Update button should show loading state

From SyncLinear.com | NV-3072

@AkshitSinghal224
Copy link

hii team, i would like to work on this issue but i am not able to open this recorded video can you help me with that.

@sudhanshugautam2911
Copy link
Contributor

yes...Unable to play video.

@jainpawan21
Copy link
Member Author

Updated description with correct video link

@iknlaxmi
Copy link
Contributor

iknlaxmi commented Nov 1, 2023

hi,I would like to contribute to this issue.

@jainpawan21
Copy link
Member Author

Assigned to you @iknlaxmi

@jainpawan21
Copy link
Member Author

@iknlaxmi how is it going?

@iknlaxmi
Copy link
Contributor

iknlaxmi commented Nov 7, 2023

@jainpawan21 Almost done today or tomorrow I will submit PR.

@iknlaxmi
Copy link
Contributor

iknlaxmi commented Nov 7, 2023

@jainpawan21 I have submitted PR. Please review.

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

Successfully merging a pull request may close this issue.

4 participants