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

Proposed fall back API for Harry Potter #84

Closed
wants to merge 1 commit into from

Conversation

jonico
Copy link
Contributor

@jonico jonico commented Mar 25, 2024

* proposed https://harry-potter-api-3a23c827ee69.herokuapp.com/ as fall back API as hp-api.onrender.com is currently suspended
@jonico
Copy link
Contributor Author

jonico commented Mar 25, 2024

@KostaSav: I noticed the main HP API is currently suspended, so I used my Heroku susbscription to boot something up. Please also tell me if I should change the main URL in the Postman collections or if you intend to unsuspend the render based hosting site again.

@KostaSav
Copy link
Owner

Hi @jonico ,

Please forgive me for the delayed response, it was a really busy period these past months.

Thank you for the pull request 😃

The Render hosting is suspended because of usage limits. I am just clarifying it, in case it looks like I am actively suspending it. Due to high usage, we have this issue every end of month and a paid Render tier is required to resolve it.

For now, I was able to re-activate the Heroku hosting (https://hp-api.herokuapp.com/) using a test branch and I have deployed the same code as used for the Render hosting, with the only difference being the URLs under index.html
So, we can add my Heroku URL as fallback for now and if we lose access to it in the future, of course we can use the one you suggested.

So, can you please edit your pull request, in order to include my Heroku URL?

I should also add that the Heroku URL you suggested contains the Render API endpoints under index.html
I have changed them in the Heroku version I deployed (https://hp-api.herokuapp.com/), but we need to find a way to configure the use of the corresponding endpoints. I have opened an issue for this (#85).

@KostaSav
Copy link
Owner

Fallback added in README through 01a9956.

@KostaSav KostaSav closed this May 31, 2024
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.

2 participants