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]作品詳細画面の通信部分を修正 #121

Open
wants to merge 10 commits into
base: develop
Choose a base branch
from

Conversation

yuuhanasaki39
Copy link
Contributor

@yuuhanasaki39 yuuhanasaki39 commented Oct 10, 2023

対応するissue

関連PR

  1. CompileSDKを上げる
  2. Compose導入とRecyclerViewの置き換え
  3. データクラスの変更
  4. 作品詳細画面の修正 ← いまここ

概要

  • 作品詳細画面で通信できるように修正(デバッグ環境のみ)

やってないこと

  • 通信部分をsealed classでラップして、UiStateで状態管理をする

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

  • app/gradle.propertiesにkeyを設定すると詳細画面でデータをとってくるようになる

  • [リファクタ] WorkRepository, WorkDetailViewModelあたり関連箇所

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

通信成功時 通信失敗時

その他

サーバーIP、ポート、トークンをapp/gradle.propertiesに切り出して参照できるよう変更しました。
各自でappモジュール直下にgradle.propertiesを作り以下のコードを設定してください。

app/gradle.properties

SERVER_IP=10.124.xx.xxx
SERVER_PORT=xxxx
USER_TOKEN="xxxxxx"

該当変更箇所

https://github.com/Funcy-ICT/Funcy_Portfolio_Android/pull/121/files#diff-51a0b488f963eb0be6c6599bf5df497313877cf5bdff3950807373912ac1cdc9R26-R34

@yuuhanasaki39 yuuhanasaki39 self-assigned this Oct 10, 2023
@yuuhanasaki39 yuuhanasaki39 added 優先度A This issue for α version bug Something isn't working labels Oct 10, 2023
@yuuhanasaki39 yuuhanasaki39 marked this pull request as ready for review October 10, 2023 09:11
@yuuhanasaki39 yuuhanasaki39 changed the title [WIP][fix]作品詳細画面の通信部分を修正 [fix]作品詳細画面の通信部分を修正 Oct 10, 2023
Copy link
Contributor

@M0710Fa M0710Fa left a comment

Choose a reason for hiding this comment

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

いいと思います!正常に動作することを確認しました!!
prの#106と被っている部分があるのでマージのときは @yuhei-yama と相談してください!

また,gradle.propertiesの件とRepository層でtokenを渡す旨も雄平に共有お願いします!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
優先度A This issue for α version bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[fix]作品詳細画面のHTTP通信できるようにする
2 participants