forked from stamp-team/chat-flutter
-
Notifications
You must be signed in to change notification settings - Fork 0
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
thoth000の修正 #51
Closed
Closed
thoth000の修正 #51
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
トークルーム一覧表示とメッセージ機能作成
ユーザの名前検索機能
もう少しcommitを分ければよかった
画像表示周りと起動画面切り替え実装
TextFieldのfixもアリ
room編集機能の制作
メインクラスとfile名の統一
This was
linked to
issues
Sep 21, 2020
Open
Open
Open
つまり、細かなfix
RoomEditControllerの作成とそれに対応したViewの作成
text change
roomの並び変え実装
既読機能の試行錯誤
Open
サーバー上の時間で管理するように変更した。
他人のデータをいじれないように変更した。
セキュリティ上の問題を修正
ScrollControllerを使用
メッセージ取得後に最新メッセージまでスクロールを実装
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
実装内容
いくつかのissueをまとめて
thoth000
のフォークリポジトリで解消した。関連issue
issueには無い問題も解決しているため、thoth000のフォークリポジトリを確認するのが良い。
(作業ごとにpull requestを分けてある。7個ほどPRがある。)
【ProgrammableGroupのissue一覧】
#13 (ページをすべて作り終えたため)
#15 (userIdを使う箇所はおそらく実装を終えている)
#27
#28
#34
#35
#37
#46
詳細内容
ルームのスクロール制御はメッセージ取得時だけでなく、いつでもリスト上部から最下部まで移動できるようにAppBarにボタンを配置した。
注意点
thoth000のフォークリポジトリのPR(2~16)をひとつずつ見ていくと、作業ごとにまとまっているので見やすいです。
実装が心配な箇所はPRのコメントに記載してあるため、コメントがある場合のみ重点的に見るのがオススメです。