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
Feature/wd3691 #36
Feature/wd3691 #36
Conversation
// import red from '@mui/material/colors/red'; | ||
|
||
// Callbacks can have any number of arguments and can have any return type. | ||
interface IProps { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I though we weren't using Typescript for this app?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We're using it for the frontend only. I think the discussion around TS was for the backend
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pretty sure it was the whole app. we havent had a chance to all come together to hash this out yet. maybe we need a developer coffee before the next scheduled one.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ShawnTurple is this true?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We are using ts in the frontend only
This is a partial PR. What is to come for this feature branch:
This PR is a little on the large side. Basically this comes down to:
Caveats: