-
Notifications
You must be signed in to change notification settings - Fork 2
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
Bugfix/TS 489 Diversity Report Change Calculations #2637
Bugfix/TS 489 Diversity Report Change Calculations #2637
Conversation
Visit the preview URL for this PR (updated for commit ac0f454): https://jac-admin-develop--pr2637-bugfix-ts-489-divers-vwx38qrq.web.app (expires Wed, 08 Jan 2025 10:47:58 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 4e92cf51659207b0ae3509dc5c40edde50edfec0 |
Visit the preview URL for this PR (updated for commit 216b113): https://jac-apply-admin-production--pr2637-bugfix-ts-489-diver-44uytrzn.web.app (expires Wed, 08 Jan 2025 15:09:17 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 2f31b7832d58b04f07d3816f45a5d09d461ded5e |
@HalcyonJAC Calculations all look good now :-) Just one thing: we don't need the % Change from previous column on the Applied tab. Is it possible to hide that column on the Applied tab only? Make the test white, would that be a quick solution? |
@nickaddy The |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great, @HalcyonJAC thx!
What's included?
Related issue: User Raised Issue BR_ADMIN_PR_000268 #489
Who should test?
✅ Product owner
✅ Developers
How to test?
Example exercise on Production
Steps:
Risk - how likely is this to impact other areas?
🟢 No risk - this is a self-contained piece of work
Additional context
Include screen grabs, video demo, notes etc.
Related permissions
Have permissions been considered for this functionality?
PREVIEW:PRODUCTION
can be OFF, DEVELOP or STAGING