From c2b18e9e0008ba9668d66c65f22cc51998618ad6 Mon Sep 17 00:00:00 2001 From: Matthew Bernhardt Date: Tue, 10 Feb 2026 16:22:09 -0500 Subject: [PATCH 1/4] Initial configuration for first deploy ** Why are these changes being introduced: There are a few things in this newly-forked repo that we want to tweak prior to deploying for the first time in Heroku. ** Relevant ticket(s): * https://mitlibraries.atlassian.net/browse/use-195 ** How does this address that need: The big change is dropping the required dyno sizes in `app.json` - we want to try and run smaller dynos, regardless of the stated minimums in the Installation Guide. Two other changes are being included: 1. Adopting our org's PR template rather than the one used by the Quepid maintainers 2. Adding a short project readme that will appear on our forked repo instead of the full Quepid readme. ** Document any side effects to this change: None, so long as this works. --- .github/PULL_REQUEST_TEMPLATE.md | 46 ++++++++++++++------------------ .github/README.md | 4 +++ app.json | 6 ++--- 3 files changed, 26 insertions(+), 30 deletions(-) create mode 100644 .github/README.md diff --git a/.github/PULL_REQUEST_TEMPLATE.md b/.github/PULL_REQUEST_TEMPLATE.md index 3556304ed..392b12cef 100644 --- a/.github/PULL_REQUEST_TEMPLATE.md +++ b/.github/PULL_REQUEST_TEMPLATE.md @@ -1,32 +1,26 @@ - +#### Developer -## Description - +- [ ] All new ENV is documented in README +- [ ] All new ENV has been added to Heroku Pipeline, Staging and Prod +- [ ] ANDI or Wave has been run in accordance to + [our guide](https://mitlibraries.github.io/guides/basics/a11y.html) and + all issues introduced by these changes have been resolved or opened as new + issues (link to those issues in the Pull Request details above) +- [ ] Stakeholder approval has been confirmed (or is not needed) -## Motivation and Context - - +#### Code Reviewer -## How Has This Been Tested? - - - +- [ ] The commit message is clear and follows our guidelines + (not just this pull request message) +- [ ] There are appropriate tests covering any new functionality +- [ ] The documentation has been updated or is unnecessary +- [ ] The changes have been verified +- [ ] New dependencies are appropriate or there were no changes -## Screenshots or GIFs (if appropriate): +#### Requires database migrations? -## Types of changes - -- [] Bug fix (non-breaking change which fixes an issue) -- [] Improvement (non-breaking change which improves existing functionality) -- [] New feature (non-breaking change which adds new functionality) -- [] Breaking change (fix or feature that would cause existing functionality to change) +YES | NO -## Checklist: - - -- [] My code follows the code style of this project. -- [] My change requires a change to the documentation. -- [] I have updated the documentation accordingly. -- [] I have read the **CONTRIBUTING** document. -- [] I have added tests to cover my changes. -- [] All new and existing tests passed. +#### Includes new or updated dependencies? + +YES | NO diff --git a/.github/README.md b/.github/README.md new file mode 100644 index 000000000..e3b1d423c --- /dev/null +++ b/.github/README.md @@ -0,0 +1,4 @@ +# Quepid for the MIT Libraries + +This repository allows the MIT Libraries to run an instance of Quepid in order to guide our development of platforms +like Unified Search and TIMDEX. diff --git a/app.json b/app.json index 239475473..d1a21ea5b 100644 --- a/app.json +++ b/app.json @@ -72,12 +72,10 @@ }, "formation": { "web": { - "quantity": 1, - "size": "standard-2x" + "quantity": 1 }, "worker": { - "quantity": 1, - "size": "standard-2x" + "quantity": 1 } }, "scripts": { From c5161b01694a11cf3106068f28668985b309ac51 Mon Sep 17 00:00:00 2001 From: Matthew Bernhardt Date: Thu, 12 Feb 2026 10:06:06 -0500 Subject: [PATCH 2/4] Add link to project readme from new local readme --- .github/README.md | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/.github/README.md b/.github/README.md index e3b1d423c..d78d39bcf 100644 --- a/.github/README.md +++ b/.github/README.md @@ -2,3 +2,7 @@ This repository allows the MIT Libraries to run an instance of Quepid in order to guide our development of platforms like Unified Search and TIMDEX. + +## See also + +The [project readme](../README.md) has a lot of useful information. \ No newline at end of file From af9d8dddcb034f78cd6b0a458f544f5b5437fc9d Mon Sep 17 00:00:00 2001 From: Matthew Bernhardt Date: Tue, 24 Feb 2026 11:18:21 -0500 Subject: [PATCH 3/4] Define job polling interval in env Currently, the application uses an env variable to define the solid cable polling interval, in order to allow deploying instances to tweak some platform behaviors without code changes. This proposal is to adopt the same strategy for the job queue polling interval. The default value (if no env variable is defined) is the same as it is now - 0.1 seconds - but if a deploying instance needs to run a little slower, this will enable that. --- config/queue.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/config/queue.yml b/config/queue.yml index 87c623c68..826fd478a 100644 --- a/config/queue.yml +++ b/config/queue.yml @@ -7,7 +7,7 @@ default: &default - queues: "*" threads: 3 processes: <%= ENV.fetch("JOB_CONCURRENCY", 1) %> - polling_interval: 0.1 + polling_interval: <%= ENV.fetch("JOB_POLLING_INTERVAL", 0.1) %> development: <<: *default From 78f578dd8d3814fb764b0d1f0dafa6a845a9d14c Mon Sep 17 00:00:00 2001 From: Matthew Bernhardt Date: Tue, 17 Mar 2026 10:27:44 -0400 Subject: [PATCH 4/4] Update db url to fall back to constructed url --- config/database.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/config/database.yml b/config/database.yml index e59ca512a..0fcc5e3f5 100755 --- a/config/database.yml +++ b/config/database.yml @@ -23,4 +23,4 @@ test: production: <<: *default - url: <%= ENV["DATABASE_URL"] %> + url: <%= ENV["DATABASE_URL"].presence || "mysql2://#{ENV["DB_USERNAME"]}:#{ENV["DB_PASSWORD"]}@#{ENV["DB_HOST"]}:3306/#{ENV["DB_NAME"]}" %>