FOUR-29107 | Disable Unsupported Retention Options in the UI Based on Tier#8727
Open
mcraeteisha wants to merge 7 commits intoepic/FOUR-29101from
Open
FOUR-29107 | Disable Unsupported Retention Options in the UI Based on Tier#8727mcraeteisha wants to merge 7 commits intoepic/FOUR-29101from
mcraeteisha wants to merge 7 commits intoepic/FOUR-29101from
Conversation
|
QA server K8S was successfully deployed https://ci-74a83b006d.engk8s.processmaker.net |
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.




Feature
Ticket: FOUR-29107
ci:deploy
This PR adds tier-based case retention options to the process edit UI. Which retention periods a user can select is driven by the CASE_RETENTION_TIER environment variable. Options for higher tiers are shown but disabled so users see the full set of options and cannot select those above their tier.
Implementation
Config (config/app.php)
case_retention_tierconfig key reads fromCASE_RETENTION_TIER(default '1').case_retention_tier_optionsconfig key:View (resources/views/processes/edit.blade.php)
allowedRetentionPeriodsis passed from config based oncase_retention_tier. Options not in this list are disabled so higher-tier options are visible but not selectable.- Changing the retention period from the dropdown opens the confirmation modal before applying the new period.
How To Test
CASE_RETENTION_POLICY_ENABLED=truein your.envfile.CASE_RETENTION_TIER=1in your.envfile.CASE_RETENTION_TIER=2in your.envfile.- The confirmation modal should display and the buttons should work as expected.
Code Review Checklist