Skip to content

fix(transloco): correct TRANSLOCO_MISSING_HANDLER token generic type#902

Merged
shaharkazaz merged 1 commit intojsverse:masterfrom
arturovt:fix/types-missing-handler
Apr 2, 2026
Merged

fix(transloco): correct TRANSLOCO_MISSING_HANDLER token generic type#902
shaharkazaz merged 1 commit intojsverse:masterfrom
arturovt:fix/types-missing-handler

Conversation

@arturovt
Copy link
Copy Markdown
Collaborator

@arturovt arturovt commented Feb 26, 2026

The InjectionToken for TRANSLOCO_MISSING_HANDLER was typed as InjectionToken<TranslocoMissingHandlerData> instead of InjectionToken<TranslocoMissingHandler>.

This caused a type mismatch when injecting the token into TranslocoService's constructor, which expects a TranslocoMissingHandler. The token now correctly reflects the shape of the value it provides.

The InjectionToken for TRANSLOCO_MISSING_HANDLER was typed as
InjectionToken<TranslocoMissingHandlerData> instead of
InjectionToken<TranslocoMissingHandler>.

This caused a type mismatch when injecting the token into
TranslocoService's constructor, which expects a TranslocoMissingHandler.
The token now correctly reflects the shape of the value it provides.
@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented Feb 26, 2026

Open in StackBlitz

@jsverse/transloco

npm i https://pkg.pr.new/jsverse/transloco/@jsverse/transloco@902

@jsverse/transloco-locale

npm i https://pkg.pr.new/jsverse/transloco/@jsverse/transloco-locale@902

@jsverse/transloco-messageformat

npm i https://pkg.pr.new/jsverse/transloco/@jsverse/transloco-messageformat@902

@jsverse/transloco-optimize

npm i https://pkg.pr.new/jsverse/transloco/@jsverse/transloco-optimize@902

@jsverse/transloco-persist-lang

npm i https://pkg.pr.new/jsverse/transloco/@jsverse/transloco-persist-lang@902

@jsverse/transloco-persist-translations

npm i https://pkg.pr.new/jsverse/transloco/@jsverse/transloco-persist-translations@902

@jsverse/transloco-preload-langs

npm i https://pkg.pr.new/jsverse/transloco/@jsverse/transloco-preload-langs@902

@jsverse/transloco-schematics

npm i https://pkg.pr.new/jsverse/transloco/@jsverse/transloco-schematics@902

@jsverse/transloco-scoped-libs

npm i https://pkg.pr.new/jsverse/transloco/@jsverse/transloco-scoped-libs@902

@jsverse/transloco-utils

npm i https://pkg.pr.new/jsverse/transloco/@jsverse/transloco-utils@902

@jsverse/transloco-validator

npm i https://pkg.pr.new/jsverse/transloco/@jsverse/transloco-validator@902

commit: 5c741e8

@shaharkazaz shaharkazaz merged commit 448e2c7 into jsverse:master Apr 2, 2026
6 checks passed
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