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

「修了プラクティス」と「修了したプラクティス」の数字がずれている #7635

Closed
SuzukaHori opened this issue Apr 4, 2024 · 8 comments
Assignees

Comments

@SuzukaHori
Copy link
Contributor

概要

個別プロフィールページ(https://bootcamp.fjord.jp/users/:id)で、右側の「修了したプラクティス」の数字と、左側の「修了プラクティス」の数字がずれています。
スクリーンショット 2024-04-04 12 03 23
他の人のプロフィールページも確認したところ、ほぼ全ての人が1~5ぐらいずれているように見えました。

期待される振る舞い

「修了したプラクティス」の数字と、「修了プラクティス」の数字が一致すること。

@kyokucho1989
Copy link
Contributor

kyokucho1989 commented Apr 5, 2024

@komagata @machida
おはようございます。数が不一致となる原因がわかりました。また、今後の方針を確認したいため書き込みます。

原因:進捗率に含まないプラクティスを設定する機能があったため

下記のPRより、プラクティス作成時に、そのプラクティスを進捗の計算に含む/含まないを設定できるようになっています。
#1479

また、下記PRによって進捗率が%表記から「x/x」 という表記に変わったため、差異がわかりやすくなっています。
#3191

方針の確認
私は以下の二つうち、どちらかの方針で進めたいと思っています。

  1. 「進捗の%に含まない~」 は考慮せずに「修了したプラクティス」の数字を「修了プラクティス」へ合わせる
  2. 進捗バーの表示を%表記に戻す(個数はそのまま)

どちらが良いでしょうか。また、これとは別の方針をとったほうが良いでしょうか。

以上です。

@komagata
Copy link
Member

komagata commented Apr 7, 2024

@kyokucho1989 なるほどです。

@machida う~ん、これどうするのがいいですかね~?
どっちかといえば1かな~ぐらいです。

@machida
Copy link
Member

machida commented Apr 8, 2024

@kyokucho1989 @komagata

少しUIに手間がかかりますが、以下のようにお願いします。

修了したプラクティス合計数96…A
修了した必須プラクティス91…B
卒業までに必要な必須プラクティス合計数108…C

とします。


進捗バーの横の数字は % の表記にしつつ、進捗バーのエリア(数字部分も含む)クリックをすると、以下の数字が表示されるようにしたいです。クリックしたら表示されるところを実装してくれたら、デザインはmachidaがやります。

A(必須: B / C)
96(必須: 91 / 198)


プロフィールページの修了プラクティスの数字は、

A(必須: B) としたいです。
96(必須: 91)

@kyokucho1989
Copy link
Contributor

@machida @komagata
ありがとうございます!
その方針で実装します!

@kyokucho1989
Copy link
Contributor

本番環境で確認ができましたので、closeします。

@kyokucho1989
Copy link
Contributor

@komagata @machida
すみません、こちらのIssueですが、ポイントが割り振られていませんでした。
ポイントの割り当てをお願いします。
個人的には2pt程度かと思います。

@komagata
Copy link
Member

@kyokucho1989 ポイントを割り当てさせていただきました~

@kyokucho1989
Copy link
Contributor

動作確認できましたのでクローズします。

@komagata komagata moved this to 完成 in bootcamp Aug 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

No branches or pull requests

4 participants