Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Heroku hotfix https://stackoverflow.com/a/73273766/2161301 #789

Closed
wants to merge 1 commit into from

Conversation

Reiszecke
Copy link

@Reiszecke Reiszecke commented Aug 9, 2022

As discussed in #788, Heroku has made some breaking changes on August 1st .

@joelbiffin has pointed to a solution at https://stackoverflow.com/a/73273766/2161301 and the error did disappear.

Idk if Heroku needs to fix stuff in their servers or if that will be the expected behavior now but this PR adds an additional script to reflect the changes Heroku expects now.

@CLAassistant
Copy link

CLAassistant commented Aug 9, 2022

CLA assistant check
All committers have signed the CLA.

@codeclimate
Copy link

codeclimate bot commented Aug 9, 2022

Code Climate has analyzed commit 3a0e65a and detected 0 issues on this pull request.

View more on Code Climate.

@michelson
Copy link
Member

Hi, thanks for this contribution; I would prefer to add this patch, though, as it seems more concise, https://stackoverflow.com/a/73289426/1542304

@Reiszecke
Copy link
Author

Hey, I will close this PR since despite the deploy working with the script you have linked, everything else doesn't seem work and I don't want to PR a potentially breaking change without being able to confirm stability.

#359 (comment)

No idea if it's just us misunderstanding the instructions or if the instructions are missing crucial steps but at this point I've spent almost a week trying to get this to run and I will probably look for commercial solutions from here.

@Reiszecke Reiszecke closed this Aug 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants