Merged
Conversation
plans details coversheet is ticked by default. Fix for issue #3063. Changes: - added a configurable property to # Check download of a plan coversheet tickbox config.x.plans.download_coversheet_tickbox_checked = false - use property to check "project details coversheet" by default or not if property set. If property is unset, the tickbox is unchecked.
…ct_sheet_tickbox_for_plan_downloads Feature #3063 - Feature to be able to configure whether the download
…variable due to Rails new middleware that prevents DNS rebinding by introducing a host whitelist
… for zeitwerk. Fixed some issues with the ContactUs controller because its constant name did not conform to Rails standards. removed Faker:Language helper because it did not conform to naming conventions and updated tests. Started trying to fix rspec tests
…t method is deprecated
add contributor to coversheet and add plan title to csv export
…eme_help do not repeat help instructions for every guidance group
Upgraded TinyMCE to V5
…veRecord::ConnectionNotEstablished exception The full error was ActiveRecord::ConnectionNotEstablished - No connection pool for 'ActiveRecord::Base' found for the 'other' role.
…rds filters in Research Output form
pluck with ampersand returns not id
…fierScheme prefix
Fix for issue #3217 regarding template search feature
Contributor form allows user to include whitespace at the beginning of ORCID
commented our Capybara.start
Fix/plan visibility
Translation.io new sync for rails6 25/10/2022
benjaminfaure
approved these changes
Oct 26, 2022
Contributor
There was a problem hiding this comment.
looking good so far !
It might be good to add a warning for people using query methods with raw SQL. It's been deprecated since Rails 5.2 and removed in Rails 6
You can see an example (and a solution) here : https://medium.com/@mitsun.chieh/activerecord-relation-with-raw-sql-argument-returns-a-warning-exception-raising-8999f1b9898a
DMP OPIDoR is impacted because we're using JSON fields with queries on the JSON data.
For example :
.pluck("data->>'keyword'")
.order("data->>'lastName', data->>'firstName'")
Contributor
Author
|
thanks @benjaminfaure and @pengyin-shan I added a note to the |
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.
Researchers
Administrators
Developers
This release includes a major upgrade of Rails (5.x to 6.1) and an upgrade from Ruby 2.6 to 2.7.6. Please review the following release notes as well as our upgrade guide and be aware that you will most likely need to adjust/modify any local customizations you may have made within your local fork of the code.
This release includes an upgrade from Ruby 2.6 to Ruby 2.7.6 and Rails 5.x to Rails 6.1!
You will need to do the following:
config/initializers/new_framework_defaults_6_1.rbDMPROADMAP_HOSTenvironment variable. Rails 6+ introduces new middleware that helps mitigate DNS rebinding attacks. This includes a hostname whitelist. The value you assign to this environment variable will be added to the whitelist.SUPPORTED_LOCALESCLIENT_LOCALESandDEFAULT_LOCALEin theconfig/initializers/translation.rbfor your instanceUpdates:
https://orcid.org// 0000-0000-0000-0000.Contributor form allows user to include whitespace at the beginning of ORCID #3212update_attributeswithupdatessinceupdate_attributeswas deprecatedJSONLinkValidatorclass file toJsonLinkValidatorrecord.errors[:model] << 'foo'torecord.errors.add(:model, 'foo')config/environments/*.rbfiles to use the newDMPROADMAP_HOSTenvironment variable to defined theRails.application.routes.default_url_optionsand newconfig.hostssetting.download_coversheet_tickbox_checkedflag to theconfig/initializers/_dmproadmap.rbinitializer that will default the Plans Download page to default that checkbox to be truedb/seeds.rbfile that is used for new installations so that it uses thefind_or_create_byinstead ofcreate!to prevent duplicates when the DB already exists.