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.

Sign up for GitHub

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

Jump to bottom

[Fix] データクラスの変数名とJSONキー名の統一 #106

Merged
merged 14 commits into from
Oct 23, 2023
Merged

[Fix] データクラスの変数名とJSONキー名の統一 #106

merged 14 commits into from
Oct 23, 2023

Conversation

yuhei-yama
Copy link
Contributor

対応するissue

概要

  • データクラスの変数名をJSONのレスポンスのキーと統一しました．

意図する動作内容（または変更点）

主な変更は以下です．細かい変更点はファイルを参照

  • 既に定義されていたデータクラス（認証，サインアップ，作品一覧，作品詳細）の変数名を変更
  • マイページデータクラスの追加
  • 変数名の変更に伴い，各ファイルへの適用

スクリーンショット（UI作成，変更時）

  • 特になし

その他

  • JSONレスポンスは現時点（7月26日）で最新のサーバー（developブランチ）を使用しています

@yuhei-yama yuhei-yama added the 優先度B This issue for β version label Jul 26, 2023
@yuhei-yama yuhei-yama self-assigned this Jul 26, 2023
@rekaBcey
Copy link
Contributor

rekaBcey commented Aug 3, 2023

WorkRepositoryにWorkDataおるで

nanairoaisu
nanairoaisu reviewed Oct 10, 2023
View reviewed changes
Copy link
Contributor

@nanairoaisu nanairoaisu left a comment

Choose a reason for hiding this comment

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

なんか編集してない部分にコメントできないからここでまとめて！

１←これ関係ないかも
FuncyApiの"auth/code"→"authcode"だな～って思ったけどそこはまた別の話…（メモ感覚）

２
ui/signup/SignupViewModel.kt
image
SignupDataの中身が古い，順番とかが．

３
ui/authentication/AuthenticationViewModel.kt
AuthData(inputCode.value!!, userId)
順番逆

４
ui/workDetail/WorkDetailViewModel.kt
image
_workの中身がworkDetailsの形になってるそうな…
※ここを直して動くようになるかは怪しい気がした

５
ui/workRegister/WorkRegisterViewModel.kt
image
WorkData→WorkDetails（れみねえと同じ指摘になっちゃう）
あと中身が更新前（？）

app/src/main/java/com/example/funcy_portfolio_android/model/data/WorkDataList.kt Outdated
@@ -1,7 +1,9 @@
package com.example.funcy_portfolio_android.model.data

data class WorkDataList(
val work_id: Int,
val workID: Int,
Copy link
Contributor

Choose a reason for hiding this comment

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

IntじゃなくてStringっぽそう
image

Copy link
Contributor

Choose a reason for hiding this comment

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

userIDとuser_nameたりてないけど，使わないならいいのか
って思ったけど
作品投稿で使いそうです

app/src/main/java/com/example/funcy_portfolio_android/model/data/WorkDataList.kt
val images: String
val thumbnail: String,
val description: String,
val icon: String,
Copy link
Contributor

Choose a reason for hiding this comment

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

「,」いらない，あっても動くんだっけ？（記憶力なし）

app/src/main/java/com/example/funcy_portfolio_android/model/data/WorkDetails.kt Outdated
val movie_url: String,
val tags: List<TagData>,
val group: String?,
val groupID: String?,
Copy link
Contributor

Choose a reason for hiding this comment

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

group…🤔
image

@M0710Fa
Copy link
Contributor

M0710Fa commented Oct 10, 2023

１←これ関係ないかも
FuncyApiの"auth/code"→"authcode"だな～って思ったけどそこはまた別の話…（メモ感覚）
 これは別の issue が上がってるはずなので今回は大丈夫です！

@yuhei-yama
Copy link
Contributor Author

プルリクのコメント一部反映しました！

コメントの4, 5に関しては，ViewModelだけでなく，Fragmentとか他のファイルにも修正範囲が広がってしまうことから
新たにissueを上げて修正した方が良いと思ったのでこのプルリクでは修正してません．

@yuhei-yama yuhei-yama merged commit 7090754 into develop Oct 23, 2023
@yuuhanasaki39 yuuhanasaki39 mentioned this pull request Oct 23, 2023
Closed
This was referenced Nov 13, 2023
Merged
Merged
Open
@yuuhanasaki39 yuuhanasaki39 mentioned this pull request Dec 18, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nanairoaisu nanairoaisu nanairoaisu left review comments

Assignees

@yuhei-yama yuhei-yama

Labels
優先度B This issue for β version
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[fix] データクラスのキー名とJSONのキー名の統一
4 participants
@yuhei-yama @rekaBcey @M0710Fa @nanairoaisu