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

言語コード、例文、例文の訳の受け取り #110

Open
wants to merge 9 commits into
base: dev
Choose a base branch
from

Conversation

hatano-yota
Copy link
Collaborator

#79

Copy link

@github-advanced-security github-advanced-security bot left a comment

Choose a reason for hiding this comment

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

ESLint found more than 10 potential problems in the proposed changes. Check the Files changed tab for more details.

} catch (error) {
if (isAxiosError(error)) {
const res = error.response as AxiosResponse<apiReturn, any>;
return {success: false, content: '', errorMessage: res.data.message, status: res.status};
const res = error.response as AxiosResponse<ApiErrorResponse, any>;

Check warning

Code scanning / ESLint

Disallow the `any` type Warning

Unexpected any. Specify a different type.
@hatano-yota
Copy link
Collaborator Author

hatano-yota commented Aug 24, 2023

ひとつTODOがありますが、おおまかな実装は完了しました。TODOに関してはUI変更が必要そうですが、現在ゆーほさんが取り組まれている追加・編集モーダルの実装、また例文のレスポンスが3つになる等で、UIに関して新たな変更がありそうなので、それに合わせて今後取り組む感じで問題ないと考えています。

@hatano-yota
Copy link
Collaborator Author

また、ゆーほさんが取り組まれている追加・編集モーダルの実装とコンフリクトが起きると思われます。

Copy link
Owner

@illionillion illionillion left a comment

Choose a reason for hiding this comment

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

@hatano-yota
自分の環境だと、元々exTransがなかったみたいなのでundefinedになってしまいます。
一旦devに切り替えて新しくカードを作ってから作業ブランチに戻る手順で同じ現象が起こるか確かめてみてください
image

例文訳の部分をバックスペース長押しするとこのような挙動になります。
編集する時は1つのテキストボックスから改行区切りで例文と訳を取得しているのが操作しにくかったりバグの原因になるからテキストボックスを2つに分けた方がいいのかも

default.mp4

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