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
UIIN-2896 Inventory app: Define and implement shortcut key for editing a quickMARC bib record #2479
Conversation
…g a quickMARC bib record
const searchParams = new URLSearchParams(location.search); | ||
|
||
searchParams.delete('relatedRecordVersion'); | ||
searchParams.append('shared', instance.shared?.toString()); |
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.
This is the second place we push shared
to URL. It might be worth to make a separate function for this.
I think that if we open a locally shared record using the shortcut, we will push shared
value to false. If we are going to fix that behavior like
const isInstanceShared = Boolean(isShared || isInstanceShadowCopy(instance?.source));
then it would be nice to do it in just one place.
}); | ||
}, [isHoldingsRecord]); | ||
|
||
const shortcuts = useMemo(() => [ |
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.
Please add a second argument to useMemo
.
Quality Gate failedFailed conditions |
Description
Added new
Ctrl+Shift+E
shortcut to open Instance/Holdings records in quickMARC editorScreenshots
chrome_TSlvM67oiK.mp4
Issues
UIIN-2896