Fresh Heroku install does not connect to Redis

[evanliewer]

two initializer files have a "if heroku?" conditional. Heroku is not respecting this. By removing the condition and leaving the code, it all works

[jagthedrummer]

Hey @evanliewer, can you describe the exact problem you were running into? Was an error being thrown, or was the if heroku? conditional evaluating to an incorrect value? I'm running a couple of BT apps in heroku and those bits of the initializers are doing what I expect.

[evanliewer]

no crashes. I used the deploy to heroku button. In the heroku console I received constant errors of not being able to connect to the redis server. Since it was a fresh install I hadn’t created any redis needs. Removing the if heroku conditional fixed it Get Outlook for iOS<https://aka.ms/o0ukef>

…

________________________________ From: Jeremy Green ***@***.***> Sent: Friday, December 13, 2024 11:20:10 AM To: bullet-train-co/bullet_train ***@***.***> Cc: evanliewer ***@***.***>; Mention ***@***.***> Subject: Re: [bullet-train-co/bullet_train] Fresh Heroku install does not connect to Redis (Issue #1802) Hey @evanliewer<https://github.com/evanliewer>, can you describe the exact problem you were running into? Was an error being thrown, or was the if heroku? conditional evaluating to an incorrect value? I'm running a couple of BT apps in heroku and those bits of the initializers are doing what I expect. — Reply to this email directly, view it on GitHub<#1802 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/ACPJQAMGGSBBRNYBKFIISB32FMXOVAVCNFSM6AAAAABTQDG5DOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNBSGA4TOMJTGA>. You are receiving this because you were mentioned.Message ID: ***@***.***>

[jagthedrummer]

If you used the deploy to heroku button it should have automatically provisioned redis for you. Did you remove it after deploying? (BT depends on redis for a number of things internally, so you'll definitely want to have a redis instance connected to your app.)

[evanliewer]

It deployed the redis great. However, For Reddit to run on her Roku, and for sidekick, there needs to be two lines of code dedicated for Roku. Those are the lines of code in the initializer, but what I learned was, they were not being initialize for her Roku when that conditional if statement was present. Get Outlook for iOS<https://aka.ms/o0ukef>

…

________________________________ From: Jeremy Green ***@***.***> Sent: Friday, December 13, 2024 11:51:59 AM To: bullet-train-co/bullet_train ***@***.***> Cc: evanliewer ***@***.***>; Mention ***@***.***> Subject: Re: [bullet-train-co/bullet_train] Fresh Heroku install does not connect to Redis (Issue #1802) If you used the deploy to heroku button it should have automatically provisioned redis for you. Did you remove it after deploying? (BT depends on redis for a number of things internally, so you'll definitely want to have a redis instance connected to your app.) — Reply to this email directly, view it on GitHub<#1802 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/ACPJQAMJ32NTWVHOBVMXLLT2FM3F7AVCNFSM6AAAAABTQDG5DOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNBSGE4DKNZRGU>. You are receiving this because you were mentioned.Message ID: ***@***.***>

[jagthedrummer]

Ah, OK, I think I'm understanding now. Can you run heroku config on your app and see if there's a config for HEROKU_APP_NAME? If not, are there any other HEROKU_ configs present?