New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Frontend cleanup #3444
Frontend cleanup #3444
Conversation
@metabase/core-developers needs review |
Is this part of #927 👴 ? #927 has officially made it onto the page of oldest open issues |
8e86827
to
e665262
Compare
…the-final-angular-frontier
Looks pretty good to me. One question I had was for the Flow types for some of the libraries like Underscore, Icepick etc are those our defs or community ones? If they're our custom ones we might want to consider trying to use the underscore ones at https://github.com/flowtype/flow-typed/ since I'm imagining those are a bit more vetted. |
Yeah some of them are based on the flowtype ones, it was just easiest to copy them over manually. Using their CLI tool is something to look into though. |
Remove Angular once and for all
eca767e
to
f014b92
Compare
query_builder/*.jsx
components intoquery_builder/components
like the rest of the app.