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

[Fix] データクラスおよびリサイクラービューの修正 #138

Draft
wants to merge 8 commits into
base: develop
Choose a base branch
from

Conversation

yuhei-yama
Copy link
Contributor

@yuhei-yama yuhei-yama commented Dec 5, 2023

対応するissue

概要

  • データクラスの変数名をJSONのレスポンスと一致するように修正
  • データの受け渡しの処理を変更(下記にて詳細を記載)
  • データクラスの変数名の変更に伴ってリサイクラービューおよび周辺の変数を変更

意図する動作内容(または変更点)

このプルリクでは以下の動作確認を実施しました.

  • ローカル環境のサーバーに投稿された作品がメイン画面(作品一覧)で読み込みができていること

スクリーンショット(UI作成,変更時)

画像が表示されている箇所とそうでない箇所があると思いますが
この部分についてはその他で詳細に記載しています.

その他

データの受け渡し変更について

拡張関数の追加によってリポジトリ層で
WorkDataList -> WorkDataのListにレスポンスを変換することとした.
これによって,UIレイヤ(ViewやFragment)でのデータ変更を抑えることが可能

スクリーンショットで表示されている箇所とそうでない箇所の差について

  • 表示されない原因

    • 調査中です...
  • 現状わかっていること

    • 表示されている画像はインターネット上から読み込みが可能な画像
      (今回はこれを使用しました:TheCatAPI
    • 表示されていない画像は,ローカル環境のファイルサーバーにアップロードした画像

このPRをマージするタイミングについて

  • RevertしたPRを再びマージした後で良いのかな?(要相談)

@yuhei-yama yuhei-yama self-assigned this Dec 5, 2023
@yuhei-yama yuhei-yama added 優先度B This issue for β version bug Something isn't working labels Dec 5, 2023
@yuhei-yama yuhei-yama changed the title [FIx] データクラスおよびリサイクラービューの修正 [Fix] データクラスおよびリサイクラービューの修正 Dec 5, 2023
@mrs1669 mrs1669 requested a review from M0710Fa June 13, 2024 14:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
優先度B This issue for β version bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[fix] データクラスのキー名とJSONのキー名の統一
1 participant