Skip to content

Airlock Notifier: Provide a link to the request in the UI in the email#2754

Merged
tanya-borisova merged 15 commits into
mainfrom
tborisova/pass-in-request-url-in-email
Oct 21, 2022
Merged

Airlock Notifier: Provide a link to the request in the UI in the email#2754
tanya-borisova merged 15 commits into
mainfrom
tborisova/pass-in-request-url-in-email

Conversation

@tanya-borisova

@tanya-borisova tanya-borisova commented Oct 19, 2022

Copy link
Copy Markdown
Contributor

Resolves #2756

What is being addressed

Provide a link to the Airlock request in the notification email for the user to easily navigate to the request.

How is this addressed

  • Pass in TRE url to the airlock notifier bundle
  • Change short_workspace_id to workspace_id to be able to construct a link to the Airlock request

@github-actions

github-actions Bot commented Oct 19, 2022

Copy link
Copy Markdown

Unit Test Results

518 tests   518 ✔️  13s ⏱️
    1 suites      0 💤
    1 files        0

Results for commit f3f9c84.

♻️ This comment has been updated with latest results.

Comment thread api_app/_version.py Outdated
Comment thread templates/shared_services/airlock_notifier/template_schema.json

@martinpeck martinpeck left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Comment thread templates/shared_services/airlock_notifier/template_schema.json
Comment thread api_app/event_grid/event_sender.py
@tanya-borisova tanya-borisova enabled auto-merge (squash) October 21, 2022 08:51
@tanya-borisova

Copy link
Copy Markdown
Contributor Author

/test

@github-actions

Copy link
Copy Markdown

🤖 pr-bot 🤖

🏃 Running tests: https://github.com/microsoft/AzureTRE/actions/runs/3296155413 (with refid 9743506b)

(in response to this comment from @tanya-borisova)

@tanya-borisova tanya-borisova merged commit 79397a9 into main Oct 21, 2022
@tanya-borisova tanya-borisova deleted the tborisova/pass-in-request-url-in-email branch October 21, 2022 10:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add a link to the UI screen for Airlock Request to the notification email

4 participants