DMPbelgium / roadmap

5 stars 1 forks source link

Add funder via 'project details' page #22

Closed laurastandaert closed 2 years ago

laurastandaert commented 2 years ago

What version of the DMPRoadmap code are you running? (e.g. v2.2.0) v3.0.2

Expected behaviour: User can update, change or add a funder in the project details tab of the DMP

Actual behaviour:

When a new plan is generated (since update to roadmap) based on a funder template, the funder is automatically mentioned on the project details tab and cannot be adjusted. This is not always accurate, since some funders allow to use templates by other funders, while the information on the project details tab is thus incorrect. eg FWO template for BOF funding. See also v3.0.4 release notes linked below
users can add funders through free text; and this generates new unmanaged organisations in the admin list of organisations. The previously created funders are not displayed to the end-user and the user is therefore not encouraged to reuse the same phrasing. It is not a controlled list
certain funder entries generate an error (form cannot be updated) eg 'BOF' on production; although in preview this does not throw an error. Unclear why!

Updates by roadmap team: https://www.dcc.ac.uk/blog/new-dmponline-v304-release-update-notes regarding this issue: https://github.com/DMPRoadmap/roadmap/pull/3029

laurastandaert commented 2 years ago

Belangrijk om te weten in v3.0.4:

Kolommen plan.funder_name en plan.grant_number worden verwijderd. Dat waren losse kolommen die in de tabel van de plannen werden bewaard. Zie deze lijnen. Reden hiervoor is dat ze al lange tijd vervangen waren door respectievelijk een organisatie van het type funder (via een relatie), en door een identifier (via een relatie). Dat verklaart waarom er nieuwe organisaties (managed: false) worden aangemaakt telkens iemand zelf de funder van een plan invult: het wordt niet meer bewaard bij het plan zelf.

Als we de te selecteren organisaties in de selectbox beperken, dan kan niemand nog een funder zelf invullen. In de versie van productie is het wel zo ingesteld dat er enkel een vaste lijst wordt getoond, maar als je iets ander intypt, dan resulteert dat toch in de aanmaak van een nieuwe organisatie. Een geluk bij ongeluk dus.

laurastandaert commented 2 years ago

Also related: new functionality for super admin to manage list of organisations: https://github.com/DMPRoadmap/roadmap/pull/2763 (v3.0.3)

laurastandaert commented 2 years ago

Management of list of organisations to be tackled by new functionality from common code base Error to update form in new issue: #38