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

#2544 Custom report data overflow - Make applied checked on page load #2561

Merged
merged 1 commit into from
Sep 17, 2024

Conversation

drieJAC
Copy link
Contributor

@drieJAC drieJAC commented Sep 12, 2024

What's included?

Did some investigation work for @nickaddy.
The only code change was to ensure the applied status is checked on page load

Closes #2544

Who should test?

✅ Product owner
✅ Developers
✅ UTG

How to test?

When you go to the custom report page the 'Applied' Application status should be checked by default

Risk - how likely is this to impact other areas?

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

Related permissions

Have permissions been considered for this functionality?

  • No permission changes required

PREVIEW:DEVELOP
can be OFF, DEVELOP or STAGING

Copy link

github-actions bot commented Sep 12, 2024

Visit the preview URL for this PR (updated for commit 68d876e):

https://jac-admin-develop--pr2561-feature-2544-set-app-sdlgnac9.web.app

(expires Sat, 12 Oct 2024 15:24:51 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 4e92cf51659207b0ae3509dc5c40edde50edfec0

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.

Thx, @drieJAC. We will be doing a deeper dive into the Custom report in the coming months and hoping to gather more feedback from users.

@drieJAC drieJAC merged commit 367d460 into main Sep 17, 2024
9 checks passed
@drieJAC drieJAC deleted the feature/2544-set-applied-as-default branch September 17, 2024 09:09
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.

Custom report data overflow
4 participants