Our caching to MLH wasn't working as expected. Changes in this PR should ensure the API call is cached properly.
Test process
I tested this by looking at local cache in tmp/cache and ensuring that Airtable and MLH keys/entries were created. To configure that, you can tweak the settings in development.rb, swapping config.cache_store = :file_store, "#{root}/tmp/cache/" in place of config.cache_store = :dalli_store, *ENV['DALLI_SERVER'].
Requirements to merge
[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code in hard-to-understand areas
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
Description
Our caching to MLH wasn't working as expected. Changes in this PR should ensure the API call is cached properly.
Test process
I tested this by looking at local cache in
tmp/cache
and ensuring that Airtable and MLH keys/entries were created. To configure that, you can tweak the settings indevelopment.rb
, swappingconfig.cache_store = :file_store, "#{root}/tmp/cache/"
in place ofconfig.cache_store = :dalli_store, *ENV['DALLI_SERVER']
.Requirements to merge