Skip to content

[ENG-8958] missing metadata (dcterms:type and osf:hasOsfAddon)#11309

Merged
adlius merged 2 commits intoCenterForOpenScience:feature/pbs-25-16from
aaxelb:fix/8958-validate-resourceTypeGeneral
Sep 16, 2025
Merged

[ENG-8958] missing metadata (dcterms:type and osf:hasOsfAddon)#11309
adlius merged 2 commits intoCenterForOpenScience:feature/pbs-25-16from
aaxelb:fix/8958-validate-resourceTypeGeneral

Conversation

@aaxelb
Copy link
Contributor

@aaxelb aaxelb commented Sep 16, 2025

Purpose

metadata missing from search (and institutional dashboard) -- resourceNature (dcterms:type, used for datacite's resourceTypeGeneral) and osf:hasOsfAddon should be there, but aren't

Changes

  • change serializer field resource_type_general to a ChoiceField to prevent invalid values from frontend
    • move DATACITE_RESOURCE_TYPES_GENERAL controlled vocab to a more sensible place for reuse
  • update log_gv_addon task (run after addons changed) to call node.update_search() for public nodes

QA Notes

Please make verification statements inspired by your code and what your code touches.

  • Verify changes to custom "resource type" reflected in institutional dashboard and in resourceTypeGeneral in datacite xml osf.example/metadata/<osfid>/?format=datacite-xml
  • Verify adding/removing addons reflected in institutional dashboard "addons" column (and/or hasOsfAddon property at staging-share.osf.io/trove/index-card-search?acceptMediatype=application%2Fvnd.api%2Bjson&cardSearchText[identifier]=<osfid>

Ticket

ENG-8958

@aaxelb aaxelb force-pushed the fix/8958-validate-resourceTypeGeneral branch from ccaae86 to 09e6696 Compare September 16, 2025 14:37
@aaxelb aaxelb marked this pull request as ready for review September 16, 2025 15:34
@adlius adlius merged commit 7a90a68 into CenterForOpenScience:feature/pbs-25-16 Sep 16, 2025
13 of 14 checks passed
@aaxelb aaxelb deleted the fix/8958-validate-resourceTypeGeneral branch September 16, 2025 16:57
adlius added a commit that referenced this pull request Sep 17, 2025
* [ENG-8514] Remove CSRF protection from reset password api v2 POST (#11246)

* remove csrf protection from reset password endpoint

* update test

* add background color prop to Brand (#11254)

* fix flaky test_serialized_metadata

* fix throttle test

* reset throttle cache

* don't ignore components when create a view-only link

* fixed updating subscribe_osf_general_email subscription

* [ENG-7277] Update doc to include version as relationship (#11262)

* added tests

* [ENG-8691] Wrong server on reset password email (#11271)

* use same logic for admin and web password reset

* fix tests

* enable filtering for linked-nodes endpoint (#11273)

* remove deleted users from institutional dashboard (#11261)

* fixed tests

* [ENG-7803] Update text on VOL modal for registration (#11279)

* added description filtering for preprints (#11294)

## Purpose

User should be able to filter preprints by their description

## Changes

Added description to filterable fields, added tests

## Ticket

https://openscience.atlassian.net/browse/ENG-8921?atlOrigin=eyJpIjoiNmY1MDUwZjI0Mjg2NDc2OGI2N2U1ZDQ0MjIwNzU0OTAiLCJwIjoiaiJ9

* [ENG-8689] - Fix/eng 8689 (#11267)

* Fix color picker for brands

* Remove unnecessary exclusion

* fix: file dataset metadata download (#11296)

* fix linting

* merge migrations

* [ENG-8958] missing metadata (dcterms:type and osf:hasOsfAddon) (#11309)

* fix: validate resource_type_general in api

* fix: update search after updating addons

* fix: file dataset metadata with mononym creators (#11297)

* Revert "fix: file dataset metadata with mononym creators (#11297)" (#11311)

This reverts commit 21090b2.

---------

Co-authored-by: antkryt <ant.krytskyi@gmail.com>
Co-authored-by: John Tordoff <Johnetordoff@users.noreply.github.com>
Co-authored-by: Ihor Sokhan <isokhan@exoft.net>
Co-authored-by: Vlad0n20 <137097005+Vlad0n20@users.noreply.github.com>
Co-authored-by: abram axel booth <aaxelb@users.noreply.github.com>
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.

2 participants