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

Feature/2260 Create Deployment Report #2348

Merged
merged 3 commits into from
Mar 26, 2024

Conversation

HalcyonJAC
Copy link
Contributor

@HalcyonJAC HalcyonJAC commented Mar 13, 2024

What's included?

Closes #2260

Note: This PR needs changes in digital-platform: Feature/admin 2260 Create Deployment Report.

Who should test?

✅ Product owner
✅ Developers
✅ UTG

How to test?

Example exercises:

  1. Go to the report page and check if there is a new link "Deployment" under "Handover" in the reports menu.
  2. Go to the "Deployment" report page.
  3. Check if the "Refresh" button works correctly.
  4. Check if the "Export data" button works correctly.
  5. Open the report and check if the information is present correctly.

Risk - how likely is this to impact other areas?

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

Additional context

Demo:

video-25944245-88c58148d1c4ccba2b83e573ca34bd55.mp4

Related permissions

Have permissions been considered for this functionality?

  • No permission changes required
  • Permissions have been added / updated. Details:

PREVIEW:DEVELOP
can be OFF, DEVELOP or STAGING

Copy link

github-actions bot commented Mar 13, 2024

Visit the preview URL for this PR (updated for commit 00f5238):

https://jac-admin-develop--pr2348-feature-2260-create-k8ed2wp8.web.app

(expires Thu, 18 Apr 2024 12:07:58 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 4e92cf51659207b0ae3509dc5c40edde50edfec0

@HalcyonJAC HalcyonJAC force-pushed the feature/2260-create-new-deployment-report branch from a31c257 to 00f5238 Compare March 19, 2024 12:04
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.

No Issues to report - all good

@warrensearle warrensearle removed their request for review March 25, 2024 14:07
@nickaddy
Copy link
Contributor

@HalcyonJAC Feedback from Matt: The data is of course limited but this looks spot on. Will be good to see how it looks IRL.

Copy link
Contributor

@nickaddy nickaddy left a comment

Choose a reason for hiding this comment

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

👍

@HalcyonJAC HalcyonJAC merged commit 59fd006 into main Mar 26, 2024
6 checks passed
@HalcyonJAC HalcyonJAC deleted the feature/2260-create-new-deployment-report branch March 26, 2024 14:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Create new Deployment report
5 participants