search

doubleunion / arooo

Membership management for hacker/makerspaces
GNU General Public License v3.0
66 stars 55 forks source link

Create storage.yml to resolve application error #770

Closed brittag closed 1 year ago

brittag commented 1 year ago

What github issue is this PR for, if any?

None, but our app isn't working right now.

Bugsnag email says:

Couldn't find Active Storage configuration in /tmp/build_cad5e0bc/config/storage.yml

StackOverflow says that adding this file worked for somebody else (https://stackoverflow.com/questions/52978340/heroku-couldnt-find-active-storage-configuration-in-app-config-storage-yml-r).

What does this code do, and why?

How is this code tested?

Ideally somebody else should test this before merging.

Are any database migrations required by this change?

Are there any configuration or environment changes needed?

Screenshots please :)

anaulin commented 1 year ago

Perhaps the safest way to deal with this error would be to roll back the app to a version that was able to in fact start up. Could be done via Heroku's UI, or in GitHub, reverting the most recently-merged PRs (starting with the Dependabot PRs merged in yesterday).

On Fri, Feb 3, 2023 at 9:25 AM Britta @.***> wrote:

What github issue is this PR for, if any?

None, but our app isn't working right now.

Bugsnag email says:

Couldn't find Active Storage configuration in /tmp/build_cad5e0bc/config/storage.yml

StackOverflow says that adding this file worked for somebody else ( https://stackoverflow.com/questions/52978340/heroku-couldnt-find-active-storage-configuration-in-app-config-storage-yml-r ). What does this code do, and why? How is this code tested?

Ideally somebody else should test this before merging. Are any database migrations required by this change? Are there any configuration or environment changes needed? Screenshots please :)

You can view, comment on, or merge this pull request online at:

https://github.com/doubleunion/arooo/pull/770 Commit Summary

File Changes

(1 file https://github.com/doubleunion/arooo/pull/770/files)

Patch Links:

— Reply to this email directly, view it on GitHub https://github.com/doubleunion/arooo/pull/770, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABTK4XIUQ5QXIXQVROXHCNDWVU5QFANCNFSM6AAAAAAUQQF5OU . You are receiving this because you are subscribed to this thread.Message ID: @.***>

anaulin commented 1 year ago

Looking at the #du-app-bots channel, these two things happened in suspiciously close proximity:

image
anaulin commented 1 year ago

Superceded by https://github.com/doubleunion/arooo/pull/771