Skip to content
This repository was archived by the owner on May 29, 2025. It is now read-only.

Add notifications sdk v2 sample notebook to sample-notebooks#799

Open
katyrochford wants to merge 3 commits intofinbourne:developfrom
katyrochford:master
Open

Add notifications sdk v2 sample notebook to sample-notebooks#799
katyrochford wants to merge 3 commits intofinbourne:developfrom
katyrochford:master

Conversation

@katyrochford
Copy link

Pull Request Checklist

  • Changes follow the style guide
  • Tests pass
  • Raised the PR against the develop branch
  • Notebook outputs do not contain any privileged data

Description of the PR

Describe the code changes for the reviewers, explain the solution you have provided and how it fixes the issue

This is a new notebook plus associated files which demonstrates how to setup an email notification with an html body which includes more information than you initially receive from the event body

* DEV-14806: Capitalise sides for orders-management data

* DEV-14806: Capitalise side in compliance data

* DEV-14806: Undo extra commas

* DEV-14806: Undo number change
@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

…trate how to produce an email notification containing more information than the original event body
@hanialsibai
Copy link
Contributor

/LGTM

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments