-
Notifications
You must be signed in to change notification settings - Fork 9
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
UIREC-279: Use number generator for barcode call number and accession number #633
base: master
Are you sure you want to change the base?
UIREC-279: Use number generator for barcode call number and accession number #633
Conversation
…add tests for ItemFields and TitleReceiveList
…fix TitleReceive.test, cleaning
Jest Unit Test Results305 tests +23 303 ✅ +23 3m 11s ⏱️ +11s Results for commit a3782f8. ± Comparison against base commit 292d263. This pull request removes 1 and adds 24 tests. Note that renamed tests count towards both.
♻️ This comment has been updated with latest results. |
…callNumber-accessionNumber
…fix TitleDetailsReceivedActions.test.js
…add transformIgnorePatterns, disable test
…add missing dependencies
…more tests for NumberGeneratorModal
…adapt test for NumberGeneratorModal
@usavkov-epam , @alisher-epam, @SerhiiNosko Sorry for the missing 2.2% code coverage. |
…js add test for NumberGeneratorModal
|
you can found uncovered lines using sonar report, here is page for it |
@SerhiiNosko Yeah, I know where the uncoverd lines are. The problem is how to cover them :/ |
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 good for me, but please wait review from @usavkov-epam also
UIREC-279
Purpose
This PR is based on the former PR from @EthanFreestone
https://github.com/folio-org/ui-receiving/pull/506/files#diff-fbeee69d56db2e9075b8c4032f3ab5656a02f3848ad42123d79d7a156f7736d2
UIOR-1383: Settings for number generator ui-orders#1696
1. add the Number generator settings in ui-orders:
settings -
orders
- General: Number generator optionsAt least one number has to be "On, field editable" or "On, field not editable"
2. Create a order to receive:
Approach
Screenshots
Pre-Merge Checklist
Before merging this PR, please go through the following list and take appropriate actions.
If there are breaking changes, please STOP and consider the following:
Ideally all of the PRs involved in breaking changes would be merged in the same day to avoid breaking the folio-testing environment. Communication is paramount if that is to be achieved, especially as the number of intermodule and inter-team dependencies increase.
While it's helpful for reviewers to help identify potential problems, ensuring that it's safe to merge is ultimately the responsibility of the PR assignee.