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-381 Remove default 100 field for Create authority, fix initial values for 008 (follow-up) #632

Merged
merged 2 commits into from
Dec 11, 2023

Conversation

BogdanDenis
Copy link
Contributor

Description

Removed default 100 field when creating Authority records
Fixed default values for 008 field. Need to take values from 008 field specification endpoint. This makes the form pristine when page is loaded

Screenshots

chrome_wqxcZfwvve.mp4

Issues

UIQM-381

@BogdanDenis BogdanDenis requested review from Dmytro-Melnyshyn and a team December 8, 2023 12:28
Copy link

github-actions bot commented Dec 8, 2023

Jest Unit Test Statistics

    1 files  ±0    39 suites  ±0   2m 14s ⏱️ -3s
324 tests ±0  324 ✔️ ±0  0 💤 ±0  0 ±0 
325 runs  ±0  325 ✔️ ±0  0 💤 ±0  0 ±0 

Results for commit 53ca851. ± Comparison against base commit 2facb13.

♻️ This comment has been updated with latest results.

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 0 Code Smells

100.0% 100.0% 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 00220d8 into master Dec 11, 2023
@BogdanDenis BogdanDenis deleted the UIQM-381 branch December 11, 2023 10:54
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