Skip to content
This repository has been archived by the owner on Jun 25, 2020. It is now read-only.

fix: extract css per package to style.css #56

Closed
wants to merge 7 commits into from
Closed

Conversation

kristw
Copy link
Collaborator

@kristw kristw commented Apr 12, 2019

An alternative solution to #55

@kristw kristw requested a review from a team as a code owner April 12, 2019 05:37
@kristw
Copy link
Collaborator Author

kristw commented Apr 12, 2019

mmm. still broken when building production storybook

@kristw kristw added the WIP label Apr 12, 2019
@kristw kristw closed this Apr 26, 2019
nytai pushed a commit to preset-io/superset-ui-plugins that referenced this pull request Apr 27, 2020
* [build] fix typescript builds

* [typescript] ensure types pass in build

* [typescript][connection] declare modules in tests

* [typescript][connection] fix ts errors in tests

* [typescript][connection] test/types.ts => types/external.d.ts

* [chart][typescript] add @types/react-loadable

* [chart][components] convert to ts

* [charts][tests][broken] convert to ts

* [chart][typescript] re-write component generics

* [chart][typescript] fix reactify generic, add react-dom types

* [chart][typescript] more iteration

* - Tweaking reactify types (using Readonly types).
- Uncovered an issue in which ReactifyProps and Props can collide on id and className.
- Move @types/react-loadable to dev dependency
- Fixing a lint error

* [chart][deps] add @types/fetch-mock

* [client][typescript] add and export SupersetClientInterface

* [chart][clients] fix ts

* [charts][components] more ts iterations

* [chart][client] assert FormData type

* [chart][deps] try adding newest @types/react

* [chart][components][ts] fix reactify prop TS

* [chart] lint

* [chart][ts] lint #2, move @types to deps not dev-deps

* [chart][jest] fix tests

* [chart][tests] up branch coverage

* [chart][ts][test] null => undefined

* [chart][tests] hundo

* [chart][tests] update name

* [chart][ts] ChartClient type fixes
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants