• Robert Knight's avatar
    Specify type of `state` arguments to `createSelector` calls · 7bc88d9c
    Robert Knight authored
    The newest version of the `createSelector` typings require the type of
    the input argument to be determined in order for the whole call to
    typecheck. This commit implements a pattern where:
    
    - A `State` type is added in various store modules, typically defined as
      `typeof initialState`, that defines the state of that store module's
      type.
    - The type of selector function params in `createSelector` calls is
      specified with `@param {State} state` and the return type is inferred
    
    The upside of these changes is that types of selector functions are
    determined much more robustly - previously it was possible to have
    incorrect types specified via JSDoc.
    
    In the process an existing minor bug was determined where an
    `isHighlight` call in `ThreadList` was passed an unexpected type of
    object (the `annotation` property of a `Draft`) for which `isHighlight`
    did not crash but would always return false. I have simply suppressed
    this existing issue for the moment, but it will need to be fixed
    separately.
    7bc88d9c
Name
Last commit
Last update
..
test Loading commit data...
activity.js Loading commit data...
annotations.js Loading commit data...
defaults.js Loading commit data...
direct-linked.js Loading commit data...
drafts.js Loading commit data...
filters.js Loading commit data...
frames.js Loading commit data...
groups.js Loading commit data...
links.js Loading commit data...
real-time-updates.js Loading commit data...
route.js Loading commit data...
selection.js Loading commit data...
session.js Loading commit data...
sidebar-panels.js Loading commit data...
toast-messages.js Loading commit data...
viewer.js Loading commit data...