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

UIBULKED-586 Support multiple edits to statistical codes in one bulk edit job #685

Merged
merged 5 commits into from
Feb 7, 2025

Conversation

vashjs
Copy link
Contributor

@vashjs vashjs commented Feb 5, 2025

This PR implements the ability for users to perform multiple edits on statistical codes within a single bulk edit job. The enhancement allows for both adding and removing statistical codes in a more flexible workflow.

Users can now select the Statistical code option more than once within a bulk edit job. This enables:

Multiple edit support:

  • An initial selection to Add statistical codes.
  • A subsequent selection to Remove statistical codes, and vice versa.

Action restrictions:

  • After choosing the Add action, the option remains available but only supports the Remove action on a subsequent selection.
  • After choosing the Remove action, the option remains available but only supports the Add action on a subsequent selection.
  • If the Remove All action is selected, the Statistical code option becomes disabled for any further edits.

UIBULKED-586

Refs: UIBULKED-586

This comment has been minimized.

Copy link

github-actions bot commented Feb 5, 2025

Jest Unit Test Statistics

    1 files  ±  0    54 suites  ±0   3m 14s ⏱️ -4s
361 tests +26  361 ✔️ +26  0 💤 ±0  0 ±0 
368 runs  +26  368 ✔️ +26  0 💤 ±0  0 ±0 

Results for commit 3c86646. ± Comparison against base commit f130dcc.

♻️ This comment has been updated with latest results.

This comment has been minimized.

Copy link

github-actions bot commented Feb 5, 2025

BigTest Unit Test Statistics

0 tests  ±0   0 ✔️ ±0   0s ⏱️ ±0s
0 suites ±0   0 💤 ±0 
0 files   ±0   0 ±0 

Results for commit 3c86646. ± Comparison against base commit f130dcc.

♻️ This comment has been updated with latest results.

@vashjs vashjs marked this pull request as ready for review February 6, 2025 15:16
@vashjs vashjs requested a review from a team February 6, 2025 15:16
@NikitaSedyx NikitaSedyx requested a review from a team February 7, 2025 10:05
@vashjs vashjs merged commit 22fd813 into master Feb 7, 2025
6 checks passed
@vashjs vashjs deleted the UIBULKED-586 branch February 7, 2025 11:29
Copy link
Member

@zburke zburke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm late to the review party here, but I just wanted to say thank you for the helpful and thorough documentation in the PR description and function headers.

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.

4 participants