Stop prefixing character avatars with the legacy HotUtils image host - #117
Open
hotutils wants to merge 2 commits into
Open
Stop prefixing character avatars with the legacy HotUtils image host#117hotutils wants to merge 2 commits into
hotutils wants to merge 2 commits into
Conversation
The optimizer's backend now serves character definitions from HotUtils V2, whose baseImage is an absolute URL on HotUtils' public image mirror. The relative-path branch that prefixed https://api.hotutils.com/images is dead code and that host is being switched off, so GameSettings takes the URL as given. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01UHQhYd9PioAZWHond3F1Mw
…elopment Every call to the optimizer API used the production host inline. One constant, overridable through CRA's REACT_APP_API_BASE, lets a local build talk to a local run of mods-optimizer-api (its local_server.py) -- which is how the HotUtils V2 switch was tested end to end. Production builds are unchanged. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01UHQhYd9PioAZWHond3F1Mw
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The optimizer's backend (mods-optimizer-api, MR on GitLab) now serves
/characters/from HotUtils V2, whosebaseImageis an absolute URL on HotUtils' public image mirror. ThestartsWith('/')branch inGameSettingsthat prefixedhttps://api.hotutils.com/imagesis therefore dead code, and that host is being switched off with the rest of the legacy HotUtils API — so this takesavatarUrlas given.No behaviour change for users once the backend MR is deployed; before it, legacy paths would render as broken images, so merge this after (or together with) the backend switch.
🤖 Generated with Claude Code