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

UIDATIMP-1440, UIDATIMP-1439, UIDATIMP-1438: Multiple holdings/items #1447

Merged
merged 17 commits into from
Aug 8, 2023

Conversation

mariia-aloshyna
Copy link
Contributor

@mariia-aloshyna mariia-aloshyna commented Aug 4, 2023

This PR is a combination of 3 PRs:

UIDATIMP-1440

To update the Data Import log JSON tab for items, to accommodate multiple items being created and/or updated and/or errored by importing individual MARC Bibliographic records, without adding more tabs to the JSON screen.

UIDATIMP-1438

To update the Data Import UI to accommodate multiple holdings and items being created and/or updated by importing individual MARC Bibliographic records

UIDATIMP-1439

To update the Data Import log JSON tab for holdings, to accommodate multiple holdings being created and/or updated and/or errored by importing individual MARC Bibliographic records, without adding more tabs to the JSON screen

@github-actions
Copy link

github-actions bot commented Aug 4, 2023

Jest Unit Test Statistics

       1 files  ±  0     217 suites  +11   14m 58s ⏱️ + 1m 54s
1 145 tests +31  1 145 ✔️ +31  0 💤 ±0  0 ±0 
1 156 runs  +31  1 156 ✔️ +31  0 💤 ±0  0 ±0 

Results for commit dcc5f46. ± Comparison against base commit 92cf80b.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Aug 4, 2023

BigTest Unit Test Statistics

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

Results for commit dcc5f46. ± Comparison against base commit 92cf80b.

♻️ This comment has been updated with latest results.

…ing multiple Holdings mapping (#1436)"

This reverts commit e360d19.
@mariia-aloshyna mariia-aloshyna changed the title UIDATIMP-1443, UIDATIMP-1440, UIDATIMP-1439, UIDATIMP-1438: Multiple holdings/items UIDATIMP-1440, UIDATIMP-1439, UIDATIMP-1438: Multiple holdings/items Aug 4, 2023
@sonarcloud
Copy link

sonarcloud bot commented Aug 8, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 18 Code Smells

94.7% 94.7% Coverage
0.0% 0.0% Duplication

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.

3 participants