Steps:

1. Create new branch
2. Install 3.1.4 locally using RVM (if needed)
3. Modify ruby version in Gemfile and .ruby-version (do not manually modify Gemfile.lock)
4. Run bundle update
5. Update TargetRubyVersion to 3.1 in .rubocop.yml
6. Run bundle exec rubocop. address linting errors
7. Update ruby, and bundler version in .travis.yml
8. Update ubuntu version to focal (20.04) in travis.yml
9. Push branch, see what happens

This is currently blocked by boston-library/bpluser#22

Rather than using multiple .env files, use Rails environment-specific encrypted YAML files to store credentials.

For the Newspapers from the Boston Public Library collection (commonwealth:4168b346w) show view,
remove the series facet values from the display (since these are only for issues of the Liberator)
and replace it with collection_name_ssim facet values (which will show all the collections for individual titles)

Notes:

The series facet values could be hidden using CSS, since there is a parent div with the document-specific id #doc_commonwealth-4168b346w.

The collection_name_ssim facet values should be in alphabetical order, and include a thumbnail.

The collection_name_ssim facet in the blacklight_config is modified by CommonwealthVlrEngine::ControllerOverride#relation_base_blacklight_config, which is called in a before_action in CollectionsControllerBehavior#show.

To accomplish this ticket, we may need to create a local CollectionsController file and override the default #show action to re-set the facet config before the call to facets_search_service.search_results.

Alternately, you could set a before action to fetch the list of collections and then render them as a separate collection of documents.

Possible idea:
Users who have successfully created accounts could be allowed to skip the recaptcha verification when downloading files.

This could be problematic though. We already get a lot of logged-in users creating spam folders.

Also this could be tricky to implement since user-related functionality is in a separate gem (bpluser) from download functionality (commonwealth-vlr-engine)

More thought required.

To fix mikel/mail#1538, we need to update the mail gem to 2.8.1 once it is released (which should be imminent).

For the moment, this can be fixed by making the following change to config/environments/production.rb:

config.action_mailer.sendmail_settings = {
  location:  "/usr/sbin/sendmail", arguments: ["-i"] 
}

A hotifx has been implemented in production for the moment.

(This was causing errors in production and QC, though not on Staging for some reason. Weird.)

In app/assets/stylesheets/commonwealth_vlr_engine.scss we provide a custom file name for the $header_logo_image variable.

We should make this image dependent on the environment that the app is being deployed to: staging, qc, or production.

There are several existing files in app/assets/images for each environment, we may need to standardize the file name convention to facilitate the desired behavior

dc-logo_tagline_sm_search-dev.png
dc-logo_tagline_sm_search-STAGING.png
dc-logo_tagline_sm_search-QC.png
dc-logo_tagline_sm.png (used in production)

The db/migrations and db/schema.rb files have definitions for tables that are left over from when there was a shared migrations submodule for Commonwealth-public-interface and Commonwealth_2 applications.

The following tables should be removed:

• admin_statistics
• polaris_barcodes
• polaris_lookups

Docs:

• https://postmarkapp.com/send-email/rails
• https://github.com/activecampaign/postmark-rails

Since we're now using encrypted secrets(via #40) we should move the gem 'dotenv-rails', '~> 2.7', require: 'dotenv/rails-now into the group :development, :test block

• Puma right now is set in the Gemfile like gem 'puma', '>= 3.12.4', '< 5'. this needs to be changed to gem 'puma' , '~> 5.6.5'
• Also the the file in config/puma.rb needs to be tweaked to look almost identical to this But with the following REMOVED

if defined?(Curator)
    puts 'Reloading Remote Service Connection Pools...'
    Curator::Services::RemoteService.reload!
  end

if defined?(Curator)
    Curator::Services::RemoteService.clear!
  end