Skip to content

fix(web): move hardcoded feed placeholder to i18n - #1227

Closed
Schro-code wants to merge 2 commits into
deeplethe:mainfrom
Schro-code:upstream/fix-library-feed-placeholder
Closed

Schro-code wants to merge 2 commits into
deeplethe:mainfrom
Schro-code:upstream/fix-library-feed-placeholder

Conversation

@Schro-code

Copy link
Copy Markdown

Summary

The custom source endpoint placeholder ("https://your-service/utopia-feed") was the only hardcoded English string in the source modal — all sibling fields already used S.library.* keys.

This moves it to S.library.feedPlaceholder for consistency with the rest of the form.

Changes

  • web/src/i18n/en.ts — add feedPlaceholder key
  • web/src/pages/Library.tsx — replace hardcoded string with i18n reference

Test plan

  • Placeholder text unchanged in English (same string, now from i18n)
  • Chinese translation can now override it via zh.ts

Zhoulu added 2 commits October 14, 2026 17:24
The custom source endpoint placeholder was the only hardcoded English
string in the source modal — all sibling fields already used S.library.*
keys. Move it to S.library.feedPlaceholder for consistency.
The previous commit added the key to en.ts but missed zh.ts, breaking
the type check since both bundles must be structurally identical.
@Schro-code

Copy link
Copy Markdown
Author

Closing to re-open against dev per CONTRIBUTING.md — PRs must target dev, not main.

@Schro-code Schro-code closed this Oct 14, 2026
@Schro-code
Schro-code deleted the upstream/fix-library-feed-placeholder branch October 14, 2026 15:54
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.

1 participant