Skip to content
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

Upgrade to react-md@v2 #17

Closed
10 of 12 tasks
carlgieringer opened this issue Jul 9, 2021 · 2 comments · Fixed by #606
Closed
10 of 12 tasks

Upgrade to react-md@v2 #17

carlgieringer opened this issue Jul 9, 2021 · 2 comments · Fixed by #606
Assignees
Labels
clean code Work to improve the codebase
Milestone

Comments

@carlgieringer
Copy link
Contributor

carlgieringer commented Jul 9, 2021

Also we should consider cross platform frameworks: https://docs.google.com/spreadsheets/d/1Wrpe54CDwLzlAJ2QJgiMRyQ3hAaNnGpfhtCKAJ2OurU/edit

Upgrade steps:

  • Buttons
  • Dialogs (CreateJustificationDialog and transclusion)
  • Cards
  • Progress
  • TextFields
  • Menus, Dividers
  • Switch/SelectionControlGroup
  • Chip
  • Autocomplete
  • Grid (see LoginPage)
  • App/Nav (TabsProps, Toolbar, Drawer, Snackbar, Tabs)
  • Form that displays model's ErrorMessages?
@carlgieringer carlgieringer added the clean code Work to improve the codebase label Jul 9, 2021
@carlgieringer carlgieringer added this to the P1 milestone Jul 9, 2021
@carlgieringer carlgieringer self-assigned this Oct 9, 2022
@carlgieringer carlgieringer removed their assignment Mar 4, 2023
@carlgieringer
Copy link
Contributor Author

We'll do #304 instead.

@carlgieringer
Copy link
Contributor Author

carlgieringer commented Apr 21, 2023

Actually we might upgrade react-md first. It's a good idea to convert most of the web app components to TS first so that typechecking helps us find any errors. We can use this command to find React components that haven't been converted to TS yet:

find . -type f -name "[[:upper:]]*.js"

Some items:

  • Use messageProps instead of ErrorMessages/direct FormMessage.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
clean code Work to improve the codebase
Projects
Development

Successfully merging a pull request may close this issue.

1 participant