Open alsoscotland opened 5 years ago
Thanks for updating to v4.0.0 and koa-session. any chance of adding type definitions here. I have been using the ones at definitely typed. They are a relatively small definition but I was having a bit of trouble trying to adjust them to work and get definitely-typed running locally to confirm that an update worked.
The current definitions are: https://github.com/DefinitelyTyped/DefinitelyTyped/blob/master/types/koa-redis/index.d.ts
// Type definitions for koa-redis 3.0
// Project: https://github.com/koajs/koa-redis
// Definitions by: Nick Simmons <https://github.com/nsimmons>
// Definitions: https://github.com/DefinitelyTyped/DefinitelyTyped
// TypeScript Version: 2.3
import { ClientOpts } from 'redis';
import { SessionStore } from 'koa-generic-session';
declare namespace redisStore {
interface RedisOptions extends ClientOpts {
duplicate?: boolean;
client?: any;
}
interface RedisSessionStore extends SessionStore {
client: any;
}
}
declare function redisStore(options: redisStore.RedisOptions): redisStore.RedisSessionStore;
export = redisStore;
Thanks for maintaining the project. I am curious if there is an ETA or plan to accept https://github.com/koajs/koa-redis/issues/35 https://github.com/koajs/koa-redis/pull/56
I am running into a typescript compilation error when trying to setup up a project with koa-redis/koa-session. I think the upgrade would alleviate the issue of conflicting
session
types in these two dependencies.