Adding Research Outputs (DB and model components only)#2739
Merged
xsrust merged 22 commits intoDMPRoadmap:developmentfrom Dec 8, 2020
Merged
Adding Research Outputs (DB and model components only)#2739xsrust merged 22 commits intoDMPRoadmap:developmentfrom
xsrust merged 22 commits intoDMPRoadmap:developmentfrom
Conversation
Pushing base DMPRoadmap code into new roadmap-main branch
…nto DMPRoadmap-development
ebe85e2 to
6083af4
Compare
…into research-outputs
Syncing with DMPRoadmap
add new mime_types Rake task to upgrade task reverted sym links reverted gitignore changes reverted blank lines added to bottom of a few configs reverted issue with open_aire config minor text change to comment in open_aire initializer to force rubocop fix Remove blank line added to open_aire initializer by Atom to fix rubocop
…into research-outputs
…into research-outputs
xsrust
reviewed
Dec 2, 2020
Comment on lines
+20
to
+27
| puts "\n== Copying sample files ==" | ||
| %w[ | ||
| config/database.yml.sample | ||
| config/initializers/wicked_pdf.rb.example | ||
| ].each do |file| | ||
| new_file .gsub('.sample', '').gsub('.example', '') | ||
| cp file, new_file unless File.exist?(new_file) | ||
| end |
xsrust
reviewed
Dec 2, 2020
config/initializers/open_aire.rb
Outdated
Comment on lines
+1
to
+10
| # frozen_string_literal: true | ||
|
|
||
| # These configuration settings are used to communicate with the | ||
| # Open Aire Research Project Registry API. For more information about | ||
| # the API and to verify that your configuration settings are correct, | ||
| Rails.configuration.x.open_aire.api_base_url = "https://api.openaire.eu/" | ||
| # The api_url should contain `%s. This is where the funder is appended! | ||
| Rails.configuration.x.open_aire.search_path = "projects/dspace/%s/ALL/ALL" | ||
| Rails.configuration.x.open_aire.default_funder = "H2020" | ||
| Rails.configuration.x.open_aire.active = true |
Contributor
There was a problem hiding this comment.
Which initializer should we be using for open_aire?
The one in external_apis or this one in initializers?
Contributor
Author
There was a problem hiding this comment.
oops. the one in config/initializers/external_apis/ is the correct one. Will remove this
xsrust
approved these changes
Dec 2, 2020
portagenetwork
pushed a commit
to portagenetwork/roadmap
that referenced
this pull request
Feb 24, 2022
* sym linked necessary files and ignored others * add research_outputs and mime_types * add new mime_types Rake task to upgrade task * reverted sym links * reverted gitignore changes * reverted blank lines added to bottom of a few configs * reverted issue with open_aire config * minor text change to comment in open_aire initializer to force rubocop fix * Remove blank line added to open_aire initializer by Atom to fix rubocop * add research_outputs and mime_types add new mime_types Rake task to upgrade task reverted sym links reverted gitignore changes reverted blank lines added to bottom of a few configs reverted issue with open_aire config minor text change to comment in open_aire initializer to force rubocop fix Remove blank line added to open_aire initializer by Atom to fix rubocop * updated schema date in schema.rb * removed dupplicate copy of open_aire initializer
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Part of #2738.
Changes proposed in this PR:
mime_typesandresearch_outputsbin/setupto run the new Rake task