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

ログインしている人の最終オンライン時刻が古くなっていた問題を修正 #4012

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

ログインしている人の最終オンライン時刻が古くなっていた問題を修正 #4012

wants to merge 2 commits into from

Conversation

nokhnaton
Copy link
Contributor

fix: #3893

@codecov
Copy link

codecov bot commented Jul 4, 2023 •
edited
Loading

Codecov Report

Patch has no changes to coverable lines.

📢 Thoughts on this report? Let us know!.

@nokhnaton nokhnaton self-assigned this Jul 4, 2023
mehm8128
mehm8128 reviewed Jul 5, 2023
View reviewed changes
Copy link
Member

@mehm8128 mehm8128 left a comment

Choose a reason for hiding this comment

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

オンライン中の人は今の時刻じゃなくて「オンライン」みたいな表示でいいかなーって思ってたのですがどうですか(デザインチームとなんか話したりしてましたっけ)

@nokhnaton
Copy link
Contributor Author

確かに「オンライン」みたいな表示でも良い気がしますね……
デザインチームとなにか決めた記憶は私には無いです。デザインチームに相談するべきな気がしてます

@mehm8128
Copy link
Member

mehm8128 commented Jul 5, 2023

なるほどです
そしたらデザインチームに相談してみるのがよさそう(多分割と軽めなのですぐ決めてくれそう＆軽く決めてもらうのがよさそう)

ras0q
ras0q requested changes Nov 30, 2023
View reviewed changes
Copy link
Member

@ras0q ras0q left a comment

Choose a reason for hiding this comment

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

オンラインのデザイン変わったので最新にするとよさそう
last-onlineで時刻にパースしてるせいで表示バグってる
image

@mehm8128
Copy link
Member

これ多分デザインチームでそこそこ大きく変更を検討してるので一旦放置でいい気がしています

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mehm8128 mehm8128 mehm8128 left review comments

@ras0q ras0q ras0q requested changes

Requested changes must be addressed to merge this pull request.

Assignees

@nokhnaton nokhnaton

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

オンライン中の最終オンライン日時を常に現在の時刻にする
3 participants
@nokhnaton @mehm8128 @ras0q