wso2 / product-apim

Welcome to the WSO2 API Manager source code! For info on working with the WSO2 API Manager repository and contributing code, click the link below.
http://wso2.github.io/
Apache License 2.0
845 stars 785 forks source link

Improving revision update support #10769

Open tmkasun opened 3 years ago

tmkasun commented 3 years ago

Describe your problem(s)

On the current Deployments page, We don't have a mechanism to update the deployed revision with a single click / action image

This improvement will change the above view as follows and provide the Update Deployment button to update the deployed revision.

Had an offline discussion with @Krishanx92 @erangatl & @tharindu1st and came up with the following design

image

Excalidraw link

Describe your solution

@nuwand @sanjeewa-malalgoda , @bhathiya , @tgtshanika , @malinthaprasan any suggestions or comments on this approach?

erangatl commented 3 years ago

XD: https://xd.adobe.com/view/be3fc799-0d72-473d-a515-9f7665b5d08a-eb6c/screen/5dd0d6ab-e04d-4460-a3fd-632059f8a02b

image

bhathiya commented 3 years ago

what if we move the blue label and the undeploy button to the "deployed revision" column, and the dropdown and update button to the action column?

Deployed revision        |     Action
----------------------------------------------------------------
<blue label>  Undeploy   |    <dropdown>    Update
bhathiya commented 3 years ago

another suggestion is this.

Deployed revision        |     Action
----------------------------------------------------------------
<blue label>             |  Undeploy        <dropdown>    Update
Krishanx92 commented 3 years ago

another suggestion is this.

Deployed revision        |     Action
----------------------------------------------------------------
<blue label>             |  Undeploy        <dropdown>    Update

Since we need to keep the buttons in Action column I'm also +1 for this

erangatl commented 3 years ago

another suggestion is this.

Deployed revision        |     Action
----------------------------------------------------------------
<blue label>             |  Undeploy        <dropdown>    Update

image

bhathiya commented 3 years ago

Looks better. Maybe we should rename the "update" button to "deploy" too? Having 3 buttons might confuse users. wdyt?

erangatl commented 3 years ago

Looks better. Maybe we should rename the "update" button to "deploy" too? Having 3 buttons might confuse users. wdyt?

Since it is already deployed, we thought most meaningful action would be to update the deployment. Although technically it could be different, users would get the idea. WDYT?

malinthaprasan commented 3 years ago

@Krishanx92 @tmkasun