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

Case Management Page: Fixes minor issues with saved polygon filters and disbursement #35257

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

ajeety4
Copy link
Contributor

@ajeety4 ajeety4 commented Oct 24, 2024

Product Description

A confirm alert is displayed regarding clearing disbursement results before doing any of the below operations.

image

  • Clear Polygon Filter
  • Delete Saved Polygon
  • Save new polygon

Also removes below disbursement params info when there is no disbursement results on the map,
image

Technical Summary

Ticket

(FYI - The above approach for clearing disbursement results were discussed with an AE.)

Feature Flag

geospatial

Safety Assurance

Safety story

Minor changes. Local Testing done.

Automated test coverage

QA Plan

Planning for a single QA for all issues ticket for this Page. Will report back.

Rollback instructions

  • This PR can be reverted after deploy with no further considerations

Labels & Review

  • Risk label is set correctly
  • The set of people pinged as reviewers is appropriate for the level of risk of the change

@ajeety4 ajeety4 added product/feature-flag Change will only affect users who have a specific feature flag enabled Open for review: do not merge A work in progress labels Oct 24, 2024
@ajeety4 ajeety4 marked this pull request as ready for review October 24, 2024 12:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Open for review: do not merge A work in progress product/feature-flag Change will only affect users who have a specific feature flag enabled
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant