cyclops icon indicating copy to clipboard operation
cyclops copied to clipboard

Added restart button

Open s-vamshi opened this issue 1 year ago • 7 comments

closes #464

📑 Description

✅ Checks

  • [ ] I have updated the documentation as required
  • [x] I have performed a self-review of my code

ℹ Additional context

added restart button

s-vamshi avatar Aug 10 '24 16:08 s-vamshi

@petar-cvit would like to know your thought that if it would be better to show restart button for all 3 resources if they are present in a single module

s-vamshi avatar Aug 10 '24 16:08 s-vamshi

image

s-vamshi avatar Aug 10 '24 16:08 s-vamshi

@petar-cvit image

s-vamshi avatar Aug 11 '24 02:08 s-vamshi

@petar-cvit done

s-vamshi avatar Aug 11 '24 09:08 s-vamshi

@petar-cvit should I refresh moduleDetails page on restarting? using this line window.location.href = "/modules/" + moduleName;

s-vamshi avatar Aug 11 '24 09:08 s-vamshi

should I refresh moduleDetails page on restarting? using this line window.location.href = "/modules/" + moduleName;

No need to do it. It affects only the resource and not the whole Module

petar-cvit avatar Aug 11 '24 18:08 petar-cvit

@petar-cvit can you please review now?

s-vamshi avatar Aug 12 '24 10:08 s-vamshi

@petar-cvit can you please review this pr?

s-vamshi avatar Aug 13 '24 17:08 s-vamshi

@petar-cvit can you please review this PR?

s-vamshi avatar Aug 16 '24 07:08 s-vamshi

image error case image success case

s-vamshi avatar Aug 16 '24 14:08 s-vamshi

@petar-cvit can you please review now?

s-vamshi avatar Aug 16 '24 15:08 s-vamshi