Skip to content

added research outputs to the CSV download of the plan - #3277

Merged
benjaminfaure merged 5 commits into
developmentfrom
research-outputs-csv
Jan 30, 2023
Merged

added research outputs to the CSV download of the plan#3277
benjaminfaure merged 5 commits into
developmentfrom
research-outputs-csv

Conversation

@briri

@briri briri commented Jan 10, 2023

Copy link
Copy Markdown
Contributor

A DMPTool user reported that the research outputs were not appearing on the CSV export of the Plan. This PR adds a research outputs section to the end of the CSV.

@benjaminfaure

Copy link
Copy Markdown
Contributor

@briri The research outputs weren't showing, I added the show_research_outputs option that was missing in the as_csv function.

If it's okay for you, I'll merge this PR :)

@github-actions

Copy link
Copy Markdown
1 Warning
⚠️ There are code changes, but no corresponding tests. Please include tests if this PR introduces any modifications in behavior. \n
Ignore this warning if the PR ONLY contains translation.io synced updates.

Generated by 🚫 Danger

@briri

briri commented Jan 27, 2023

Copy link
Copy Markdown
Contributor Author
@benjaminfaure
benjaminfaure merged commit 36b6dfb into development Jan 30, 2023
@benjaminfaure
benjaminfaure deleted the research-outputs-csv branch January 30, 2023 08:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants