rill icon indicating copy to clipboard operation
rill copied to clipboard

feat: Add a notification after a project deployment

Open AdityaHegde opened this issue 1 year ago • 4 comments

This handles the case where the user deployed a fresh project. Going from invite => dashboards page will trigger a wait on deployment complete.

  1. On success we show a notification with a link to 1st dashboard.
  2. On failure we redirect to status page.

AdityaHegde avatar Aug 28 '24 12:08 AdityaHegde

The notification's copy feels a little weird:

  1. There are 6 dashboards in my project, so "Go to dashboard" doesn't refer to any particular one.
  2. The main dashboards table provides links to each of my 6 dashboards. In order to click on the notification link, my cursor would have to travel past the usual links in order to reach the one-time notification link.

What if we either:

  • Just say: "Project deployed!"
  • Take the opportunity to teach the user how to navigate the normal way, e.g.: "Project deployed! Click on one of your dashboards above".

CC @ericokuma

Nit: the notification currently includes the text "shouldRedirect" image

ericpgreen2 avatar Aug 29 '24 23:08 ericpgreen2

Thanks for the feedback @ericpgreen2!

Let me circle back with @jkhwu on this

ericokuma avatar Aug 30 '24 00:08 ericokuma

Let's push this to the next sprint.

ericokuma avatar Sep 03 '24 16:09 ericokuma

IMO this still needs some design discussion. I commented in Figma here.

ericpgreen2 avatar Sep 10 '24 20:09 ericpgreen2

Given the large # of open PRs, I'll close this for now until we have Product clarity.

ericpgreen2 avatar Nov 06 '24 22:11 ericpgreen2