Details

    • Type: Improvement
    • Status: Closed (View Workflow)
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 4.0
    • Component/s: DSpace API
    • Labels:
      None
    • Attachments:
      0
    • Comments:
      3
    • Documentation Status:
      Complete or Committed

      Description

      I made a new dataloader to use metadata from CiNii Article. This source is very usefull for Japanese repository and I think it is worth to include this dataloader to DSpace as a default setting.

      "CiNii Articles" is a search service provided by NII which enables you to search information on academic articles published in academic society journals, university research bulletins or articles included in the National Diet Library's Japanese Periodicals Index Database. See http://ci.nii.ac.jp/info/en/cinii_outline.html for details.

      There are metadata which are not included in DSpace default metadata-field-set. Now I set these metadata in bte.xml. If it is not adequate, I will comment out them.

        Attachments

          Activity

          Hide
          zuki.ebetsu Keiji Suzuki added a comment -
          Show
          zuki.ebetsu Keiji Suzuki added a comment - I sent a PR. https://github.com/DSpace/DSpace/pull/392
          Hide
          kstamatis Kostas Stamatis added a comment -

          I am definitely +1 on this.

          I had the chance to review the PR code and I have the following comments:

          1) The code fully conforms to the code we have already have for the three initial services

          2) I have made some tests and seems to fully work

          3) We can reference this PR in the documentation as an example of how someone can write or contribute a new service for the submission lookup step

          4) Even if the CiNii service is not known worldwide, we can accept the code and we can comment the service in the configuration so as not to appear by default in all installations but the interested ones can uncomment it

          Regards,
          Kostas

          PS: If the code is finally accepted, the CiNii service requires an application api key which users must provide in configuration. For the http://demo.dspace.org site we are going to need our own key which can be obtained from: https://portaltools.nii.ac.jp/developer/en/

          Show
          kstamatis Kostas Stamatis added a comment - I am definitely +1 on this. I had the chance to review the PR code and I have the following comments: 1) The code fully conforms to the code we have already have for the three initial services 2) I have made some tests and seems to fully work 3) We can reference this PR in the documentation as an example of how someone can write or contribute a new service for the submission lookup step 4) Even if the CiNii service is not known worldwide, we can accept the code and we can comment the service in the configuration so as not to appear by default in all installations but the interested ones can uncomment it Regards, Kostas PS: If the code is finally accepted, the CiNii service requires an application api key which users must provide in configuration. For the http://demo.dspace.org site we are going to need our own key which can be obtained from: https://portaltools.nii.ac.jp/developer/en/
          Hide
          kstamatis Kostas Stamatis added a comment -

          Merged here: https://github.com/DSpace/DSpace/commit/a282bf103a3a8ae78e0fe7f958e5413b0c666f71

          Needs documentation, so I leave this issue open.
          I will add the documentation shortly and then I will close the issue.

          Kostas

          Show
          kstamatis Kostas Stamatis added a comment - Merged here: https://github.com/DSpace/DSpace/commit/a282bf103a3a8ae78e0fe7f958e5413b0c666f71 Needs documentation, so I leave this issue open. I will add the documentation shortly and then I will close the issue. Kostas

            People

            • Assignee:
              Unassigned
              Reporter:
              zuki.ebetsu Keiji Suzuki
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: