Skip to content
Snippets Groups Projects

Master

Merged Benedikt Heinrichs requested to merge master into Request/b6604d11-424f-45ef-9f42-fe25f4d13bdb

Note: the Review can be carried out either by checking the index.ttl file under the changes tab or by copy-and-pasting the content of the file into the AIMS Frontend

  • Copy and paste code into AIMS Frontend. Check the form using under the metadata-form button. Does everything render correctly and do all the fields work?
  • Do the title and description match
  • Is there an English and German version of the title and description? (English is OK, but bi-lingual is preferred.)
  • Are various datatypes used? (If applicable, in some cases, only using string is entirely appropriate.)
  • Does each field have a description?
  • If pre-defined terms are used, do they make sense? (broadly speaking, e.g., dcterms:author is not used for a field named "Device".)
  • Does sh:targetClass URL match the base URL? (Applies in most but not all cases.)
  • If the base URL (line one in index.ttl) contains the GUID such as https://purl.org/coscine/ap/{GUID}/, modify this to contain the name of the profile and, if required, the grouping: https://purl.org/coscine/ap/{profileName}/ or https://purl.org/coscine/ap/{group}/{profileName}/
    • Change any other instances of the base URL to match.
  • Deploy on DSP-10 by clicking the arrows in the pipeline box at the top of the merge request–just under the the reactions. (add label accordingly when done)
  • Test on DSP-10 by creating a resource with the application profile and adding a file/linked data to the resource on the web interface (add label accordingly when done). Does everything render correctly and do all the fields work?
  • If applicable, contact submitter via service desk ticket (check box here and add "waiting" label to request).
  • Once review is complete, comment and tag Petar and Benedikt to approve and merge

For more details, see the documentation.

Merge request reports

Pipeline #1337261 passed

Pipeline passed for 8a3c88b9 on master

Merged by Benedikt HeinrichsBenedikt Heinrichs 9 months ago (May 22, 2024 7:12am UTC)

Loading

Pipeline #1342417 passed

Pipeline passed for 5d29dec9 on Request/b6604d11-424f-45ef-9f42-fe25f4d13bdb

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading