Skip to content

Commit

Permalink
Merge pull request #135 from NordicSemiconductor/fix/missing_property…
Browse files Browse the repository at this point in the history
…_type

Fix: Missing prop type for pane components
  • Loading branch information
datenreisender authored Feb 5, 2021
2 parents a291439 + 60bdcdf commit f50cc96
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 2 deletions.
5 changes: 5 additions & 0 deletions Changelog.md
Original file line number Diff line number Diff line change
@@ -1,3 +1,8 @@
## 4.17.3
### Fix
- Property `active` was missing in the TypeScript definition of the pane
components.

## 4.17.2
### Fix
- Not defining an app reducer led to an error.
Expand Down
5 changes: 4 additions & 1 deletion index.d.ts
Original file line number Diff line number Diff line change
Expand Up @@ -67,6 +67,9 @@ declare module 'pc-nrfconnect-shared' {
export const logger: winston.Logger;

// App.jsx
export interface PaneProps {
active: boolean;
}

/**
* Props for the `App` component.
Expand Down Expand Up @@ -106,7 +109,7 @@ declare module 'pc-nrfconnect-shared' {
*
* `[['Connection Map', ConnectionMap], ['Server Setup', ServerSetup]]`
*/
panes: readonly (readonly [string, React.FC])[];
panes: readonly (readonly [string, React.FC<PaneProps>])[];
/**
* Describes whether the log will show automatically when the
* application starts. Defaults to `true`.
Expand Down
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "pc-nrfconnect-shared",
"version": "4.17.2",
"version": "4.17.3",
"description": "Shared commodities for developing pc-nrfconnect-* packages",
"repository": {
"type": "git",
Expand Down

0 comments on commit f50cc96

Please sign in to comment.