Skip to content

Conversation

@briri
Copy link
Contributor

@briri briri commented Nov 20, 2020

Part of #2738.

Changes proposed in this PR:

  • Add DB tables for mime_types and research_outputs
  • Add models and specs for MimeType and ResearchOutput
  • Add Rake task to fetch latest mime types and load them into the DB
  • Update bin/setup to run the new Rake task
  • Add v3.1.0 upgrade task that runs the Rake task

briri added 10 commits November 23, 2020 11:19
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
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
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks for updating this.

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
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Which initializer should we be using for open_aire?
The one in external_apis or this one in initializers?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oops. the one in config/initializers/external_apis/ is the correct one. Will remove this

@xsrust xsrust merged commit 965b75f into DMPRoadmap:development Dec 8, 2020
@briri briri deleted the research-outputs branch April 24, 2021 13:12
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
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