Skip to content

Commit 3016d4f

Browse files
Merge pull request #840 from tighten/fix-current-strict
Fix checking current route in strict mode
2 parents 92271fe + e6fdb5f commit 3016d4f

File tree

2 files changed

+3
-1
lines changed

2 files changed

+3
-1
lines changed

src/js/index.d.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -171,7 +171,7 @@ interface ParsedQs {
171171
*/
172172
interface Router {
173173
current(): ValidRouteName | undefined;
174-
current<T extends ValidRouteName>(name: T, params?: ParameterValue | RouteParams<T>): boolean;
174+
current<T extends RouteName>(name: T, params?: ParameterValue | RouteParams<T>): boolean;
175175
get params(): Record<string, string>;
176176
get routeParams(): Record<string, string>;
177177
get queryParams(): ParsedQs;

tests/js/route.test-d.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -89,6 +89,8 @@ assertType(route().has(''));
8989
// Test router getter autocompletion
9090
assertType(route().params);
9191

92+
// These should NOT error in strict mode
93+
assertType(route().current('posts.*'));
9294
assertType(route().current('missing', { foo: 1 }));
9395

9496
// @ts-expect-error missing required 'post' parameter

0 commit comments

Comments
 (0)