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

Added new drawings to job notification emails #6994

Merged
merged 5 commits into from
Apr 21, 2023
Merged

Conversation

hotzenklotz
Copy link
Member

@hotzenklotz hotzenklotz commented Apr 20, 2023

PR adds new drawings to job notification emails in accordance with new branding.

URL of deployed dev instance (used for testing):

  • https://___.webknossos.xyz

Steps to test:

  • Trigger a successful i.e. export as a tiff
  • Upload a dataset to trigger a data uploaded email
  • Start a segmentation to trigger a segementation finished email
  • Stop the WK Worker and run a job to receive a job failed email

Successful Upload

image

Failed Job (Generic)

image

Successful Job (Generic)

image

Issues:


(Please delete unneeded items, merge only when none are left open)

@hotzenklotz hotzenklotz self-assigned this Apr 20, 2023
@hotzenklotz hotzenklotz marked this pull request as ready for review April 21, 2023 13:01
…il-images

* 'master' of github.com:scalableminds/webknossos:
  Fix handling underscores in GCS bucket name (#6998)
@@ -26,4 +26,11 @@
</div>

<p>With best regards,<br />the WEBKNOSSOS team</p>

<div style="text-align: center;">
<img src="https://static.webknossos.org/mails/email-drawing-successful-job.png"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I get an access denied message for this url.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed :-)

@@ -31,6 +31,17 @@
<p>
Did you know that we offer services for <a href="https://webknossos.org/services/alignment">dataset alignment & stitching</a> and <a href="https://webknossos.org/services/automated-segmentation">AI image segmentation</a> to kickstart your research?
</p>
<div style="text-align: center; margin-top: 10px; margin-bottom: 6px;">
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

unrelated to this pr, but in line 9 of this file, it says amongst your other dataset but it should be datasetS.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed

Copy link
Member

@philippotto philippotto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm 👍 I especially like the oops picture 😅

@hotzenklotz hotzenklotz merged commit cc30156 into master Apr 21, 2023
@hotzenklotz hotzenklotz deleted the email-images branch April 21, 2023 13:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants