Skip to content
This repository has been archived by the owner on Jul 24, 2024. It is now read-only.

Better feedback when publishing/unpublishing #2343

Closed
gashcrumb opened this issue Apr 11, 2018 · 13 comments
Closed

Better feedback when publishing/unpublishing #2343

gashcrumb opened this issue Apr 11, 2018 · 13 comments
Assignees
Labels
cat/feature PR label for a new feature group/server REST backend for managing integrations group/ui User interface SPA, talking to the REST backend group/uxd User experience (UX) designs prio/p1 The priority of a bug. p1 means high

Comments

@gashcrumb
Copy link
Contributor

This is a...


[x] Feature request
[ ] Regression (a behavior that used to work and stopped working in a new release)
[ ] Bug report  
[ ] Documentation issue or request

The problem

When publishing or unpublishing an integration, the 'Pending' spinner is not enough feedback. There's a lot of factors that play a role in how long an integration takes to publish, and some of these factors can be out of our control.

I think we should expose more information to the user so that there's more feedback, i.e. 'building', 'starting pod' etc. basically more fine-grained states as the integration transitions from unpublished to published.

Perhaps keep the 'Pending' state but then add a second deployment status field perhaps, I dunno, let's discuss...

@kcbabo @dongniwang FYI...

@gashcrumb gashcrumb added cat/feature PR label for a new feature group/server REST backend for managing integrations group/ui User interface SPA, talking to the REST backend group/uxd User experience (UX) designs prio/p1 The priority of a bug. p1 means high labels Apr 11, 2018
@pure-bot pure-bot bot added the notif/triage The issue needs triage. Applied automatically to all new issues. label Apr 11, 2018
@gashcrumb gashcrumb removed the notif/triage The issue needs triage. Applied automatically to all new issues. label Apr 11, 2018
@kcbabo
Copy link

kcbabo commented Apr 11, 2018

Would be great to have a checklist/table that updates as deployment progresses through various states, e.g.

  • Assembling
  • Building
  • Deploying
  • Starting

@gashcrumb
Copy link
Contributor Author

Yeah, I think on the integration detail page if it's deploying a checklist would be good. On the integration list page should the user happen to go there I think probably just showing the current stage of the deployment would be fine in place of Publishing/Unpublishing.

@dongniwang dongniwang self-assigned this Apr 11, 2018
@gashcrumb gashcrumb added this to the Post GA (7.1) milestone Apr 17, 2018
@chirino
Copy link
Collaborator

chirino commented Apr 17, 2018

A link to the OCP console to the thing we are waiting on would be nice too. Like pod build logs, DC events page, Pod startup logs etc.

@dongniwang
Copy link
Contributor

V2 of UX design is available via UX tracker: https://syndesisio.github.io/syndesis-ux/. Please review.

@TovaCohen
Copy link
Collaborator

This looks good to me, except it is still showing the hover help for "Building" as:
Build started, waiting on it to finish.

For the hover help, I suggest:

Assembling - Creating pod resources needed to build the integration
Building - Getting the integration ready to deploy
Deploying - Waiting for deployment of pod that will run the integration
Starting - Waiting for the integration to be ready to run
Publishing - Waiting for the pod to start running the integration

@gashcrumb
Copy link
Contributor Author

@TovaCohen cool, I'll use that in the implementation, and it'll go into the dictionary file so it'll be easy to change later on.

@kcbabo
Copy link

kcbabo commented Jun 25, 2018

Hey - just to confirm, wouldn't this be the correct order?
Assembling - Creating pod resources needed to build the integration
Building - Getting the integration ready to deploy
Deploying - Waiting for deployment of pod that will run the integration
Publishing - Waiting for the pod to start running the integration
Starting - Waiting for the integration to be ready to run

@gaughan
Copy link

gaughan commented Jun 26, 2018

Looks great, especially the log link. UX Approved by PM

@dongniwang
Copy link
Contributor

Thanks @gaughan.

@gashcrumb @kcbabo @dhirajsb @TovaCohen - can we confirm what are the stages? And what is the correct order of these stages? So I can update the design to reflect the latest changes.

@kcbabo
Copy link

kcbabo commented Jun 26, 2018

From my POV, it's what I mentioned above : Assembling, Building, Deploying, Publishing, Starting

@gashcrumb
Copy link
Contributor Author

+1 on the designs from me.

@dhirajsb
Copy link
Contributor

dhirajsb commented Jul 3, 2018

Opened PR #2947 to address this in the backend. Need to run the new state monitoring endpoint to get publishing state details by @gashcrumb for UI changes.

@dongniwang
Copy link
Contributor

See latest UX design in UX tracker. Closing this issue.

If we need to address anything new, please open a new issue.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cat/feature PR label for a new feature group/server REST backend for managing integrations group/ui User interface SPA, talking to the REST backend group/uxd User experience (UX) designs prio/p1 The priority of a bug. p1 means high
Projects
None yet
Development

No branches or pull requests

8 participants