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

ILL Borrowing module #14

Open
wants to merge 22 commits into
base: nacsis-cat-ill
Choose a base branch
from

Conversation

dongchch2016
Copy link
Contributor

included:NACSIS ILL request creation;Search Bibliographic records in NACSIS

dongchch2016 and others added 22 commits September 30, 2021 18:37
included:NACSIS ILL request creation;Search Bibliographic records in NACSIS
Fxied some bugs:
concatenated issue.
extractNacsisId() function.
Display issue.
Added table display first time.
Add table features.
add view holding and member info into expandable row.
Add request forms and view member info.
Fixed some UI issues.
Add create ILL request logic and remove some unused code.
Fix view member info UI issue.
add error property. need to add to other places using getIntegrationProfile.
all callbacks in RxJS are wrapped with try-catch blocks so it'll be caught and then sent as an error notification.
Fix some UI issues and finish creating ILL request.
keep the previous page data when going back page.
Fix some 'null' back.
Sync en.json and jp.json
Fix some translation bug and add  Japanese translation
Remove unused translation
Add translation for member info
Add translation for paginator
Fix css issue and autoClosed bug.
complete the jp.json translation.
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