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

コンポーネントの追加 #40

Merged
merged 14 commits into from
Nov 17, 2020
Merged

コンポーネントの追加 #40

merged 14 commits into from
Nov 17, 2020

Conversation

knokmki612
Copy link

@knokmki612 knokmki612 commented Nov 13, 2020

pathのaliasの追加を含んでいます

@knokmki612 knokmki612 marked this pull request as draft November 13, 2020 10:15
@knokmki612
Copy link
Author

image

作成したもの

  • TextField
  • BookForm: ブックの作成と編集で使う入力項目のブロック

@knokmki612
Copy link
Author

https://github.com/cccties/ChibiCHiLO/pull/40/checks?check_run_id=1395249516#step:5:7

ciのフォーマットのジョブのコミットするステップでコケているようですね

@kou029w
Copy link

kou029w commented Nov 16, 2020

git remoteが適切に与えられていないようでコミットの反映に失敗していますね…
actions/checkout@v2 をジョブの中で呼ぶか git コマンドで適切に指定するなどする必要あるかなぁ。
いずれにせよ(yarn format後に差分が検出されたという点を除けば)本PRとは無関係なのでGitHub Actionsのほうは直しておきます。
ご報告ありがとうございます。

const cardClasses = useCardStyles();
const inputLabelClasses = useInputLabelStyles();
const classes = useStyles();
const [language, setLanguage] = useState("ja");
Copy link
Author

@knokmki612 knokmki612 Nov 17, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Selectがちゃんと動かしたかったからSelectだけuseStateを書いているけど、ここはBookの入力項目を入れたいので、最終的には、BookのuseStateを書いて、

setBook({ ...book, language: event.target.value })

のような感じに書くのかなとは思っている

@knokmki612
Copy link
Author

@kou029w
基本的にコードを追加していっており、既存のコードを触っているわけではないとはいえ、
あんまり溜めるのも負担をおかけするので、このあたりで見ていただいた方が良いのかなと思いました!

@knokmki612 knokmki612 marked this pull request as ready for review November 17, 2020 01:44
@knokmki612
Copy link
Author

と思ったらciで何か出ていて、私が直せそうなのでもう少しだけ触ります

@kou029w kou029w self-requested a review November 17, 2020 01:46
@knokmki612
Copy link
Author

@kou029w 終わりました!
material-uiのサンプルコードはちゃんとTypeScriptのほうでみるようにしようと思いました

Copy link

@kou029w kou029w left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

const inputClasses = useInputStyles();
const [value, setValue] = useState("name-descend");
const handleChange = (event: ChangeEvent<{ value: unknown }>) => {
setValue(event.target.value as string);
Copy link
Author

@knokmki612 knokmki612 Nov 17, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ここだけなぜか ChangeEvent<HTMLInputElement> で指定すると

Type '(event: ChangeEvent<HTMLInputElement>) => void' is not assignable to type '(event: ChangeEvent<{ name?: string | undefined; value: unknown; }>, child: ReactNode) => void'.
  Types of parameters 'event' and 'event' are incompatible.
    Type 'ChangeEvent<{ name?: string | undefined; value: unknown; }>' is not assignable to type 'ChangeEvent<HTMLInputElement>'.
      Type '{ name?: string | undefined; value: unknown; }' is missing the following properties from type 'HTMLInputElement': accept, align, alt, autocomplete, and 285 more.

と言われてしまうのでstringにキャストアサーションした

Co-authored-by: Kohei Watanabe <kou029w@gmail.com>
@knokmki612 knokmki612 merged commit 7d9c5ca into next Nov 17, 2020
ties-makimura pushed a commit to LTRP-SOC5-OU/ViLOG that referenced this pull request Mar 3, 2022
…y-dialog

feat: LearnerActivityDialogの実装
ties-makimura pushed a commit to LTRP-SOC5-OU/ViLOG that referenced this pull request Apr 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants