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

Virus scanner status on releases page #2536

Merged
merged 7 commits into from
Sep 24, 2024
Merged

Conversation

tomlovesgithub
Copy link
Contributor

@tomlovesgithub tomlovesgithub commented Aug 13, 2024

What's included?

The result of runScannerTest (see digital-platform 1138) is reflected on the releases page as 'Virus Scanner functionality: '.
This is to allow us to see the results of the scheduled test of the scan functionality.

Who should test?

✅ Product owner
✅ Developers
✅ UTG

How to test?

Visit the releases page and observe the scan result.
(for developers) amend the field in the database and ensure the result is reflected.

Risk - how likely is this to impact other areas?

🟢 No risk - this is a self-contained piece of work

Additional context

image


PREVIEW:DEVELOP
can be OFF, DEVELOP or STAGING

Copy link

github-actions bot commented Aug 13, 2024

Visit the preview URL for this PR (updated for commit 80d02aa):

https://jac-admin-develop--pr2536-1194-frontend-change-szlnu0kr.web.app

(expires Thu, 26 Sep 2024 12:42:07 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 4e92cf51659207b0ae3509dc5c40edde50edfec0

Copy link
Contributor

@Franceswog Franceswog left a comment

Choose a reason for hiding this comment

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

image

No issues to report.

@tomlovesgithub tomlovesgithub merged commit 2317f28 into main Sep 24, 2024
6 checks passed
@tomlovesgithub tomlovesgithub deleted the 1194/frontend-changes branch September 24, 2024 10:42
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.

4 participants