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

Improved: condition to make the value of partial allocation updated on server when selecting promise date and also enabled partial group allocation when partial allocation is enabled(#257) #258

Merged
merged 2 commits into from
Sep 3, 2024

Conversation

ymaheshwari1
Copy link
Contributor

@ymaheshwari1 ymaheshwari1 commented Sep 3, 2024

Related Issues

#257

Short Description and Why It's Useful

  • Added case to display empty state in the rule filter card, when partial group item allocation filter is selected

On selecting promise date, the partial allocation and partial allocation group items being enabled, but when removing promise date filter the allocation are not removed.

Screenshots of Visual Changes before/after (If There Are Any)

Contribution and Currently Important Rules Acceptance

…r server when selecting promise date and also enabled partial group allocation when partial allocation is enabled(hotwax#257)
… from available filter is selected(hotwax#257)

When applying partial group item allocation filter, empty state from the rule filters card is removed even when no filter is displayed inside the filter card, thus added check to not honor partial group item allocation as a filter when checking condition for displaying empty state
@ymaheshwari1 ymaheshwari1 merged commit 8ac59fc into hotwax:main Sep 3, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant