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

UIQM-574: Allow user to select an authority file when creating a new MARC authority record #622

Merged
merged 5 commits into from
Nov 22, 2023

Conversation

BogdanDenis
Copy link
Contributor

Description

Allow user to select an authority file when creating a new MARC authority record.

In scope of this ticket we only needed to implement the 001 Authority Source File lookup button and modal. Actual handling of selected Source File will be done in scope of UIQM-576

Screenshots

chrome_TBNQc9ti1J.mp4

Issues

UIQM-574

Copy link

github-actions bot commented Nov 21, 2023

Jest Unit Test Statistics

    1 files  ±  0    39 suites  +3   2m 21s ⏱️ +12s
320 tests +11  320 ✔️ +11  0 💤 ±0  0 ±0 
321 runs  +11  321 ✔️ +11  0 💤 ±0  0 ±0 

Results for commit 242ebab. ± Comparison against base commit ff5d373.

♻️ This comment has been updated with latest results.

@BogdanDenis BogdanDenis requested review from Dmytro-Melnyshyn and a team November 22, 2023 09:01
@Dmytro-Melnyshyn Dmytro-Melnyshyn requested a review from a team November 22, 2023 09:42
Copy link

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 1 Code Smell

94.3% 94.3% Coverage
0.0% 0.0% Duplication

warning The version of Java (11.0.17) you have used to run this analysis is deprecated and we will stop accepting it soon. Please update to at least Java 17.
Read more here

@BogdanDenis BogdanDenis merged commit b556cb0 into master Nov 22, 2023
4 checks passed
@BogdanDenis BogdanDenis deleted the UIQM-574 branch November 22, 2023 15:05
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.

2 participants