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

ユーザーの情報が入力されていないときの表示を修正 #245

Merged
merged 2 commits into from
Nov 17, 2024

Conversation

Kaki256
Copy link
Contributor

@Kaki256 Kaki256 commented Nov 17, 2024

User description

fix #205


PR Type

Enhancement


Description

  • ユーザーの実績が登録されていない場合の表示メッセージを更新しました。
  • ユーザーが参加したプロジェクトが登録されていない場合の表示メッセージを更新しました。

Changes walkthrough 📝

Relevant files
Enhancement
ContestsContainer.vue
Update message for unregistered achievements                         

src/components/User/ContestsContainer.vue

  • Updated message for when no achievements are registered.
+1/-1     
ProjectsContainer.vue
Update message for unregistered projects                                 

src/components/User/ProjectsContainer.vue

  • Updated message for when no projects are registered.
+1/-1     

💡 PR-Agent usage: Comment /help "your question" on any pull request to receive relevant information

Copy link

Copy link

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

🎫 Ticket compliance analysis ✅

205 - Fully compliant

Fully compliant requirements:

  • ユーザーの情報が入力されていないときに、その状態が分かるような表示を実装する。
⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ No major issues detected

Copy link

PR Code Suggestions ✨

Explore these optional code suggestions:

CategorySuggestion                                                                                                                                    Score
Enhancement
ユーザーに対するメッセージの明確化

v-elseの文言をより明確にするために、ユーザーが理解しやすい言葉を使用してください。

src/components/User/ContestsContainer.vue [40]

-<p v-else>実績はまだ登録されていません</p>
+<p v-else>現在、登録された実績はありません。</p>
Suggestion importance[1-10]: 6

Why: The suggestion improves the clarity of the message displayed to the user when no contest results are registered. It enhances user understanding by using clearer language.

6
表現の一貫性の向上

v-elseの文言を一貫性を持たせるために、他の部分と同様の表現を使用してください。

src/components/User/ProjectsContainer.vue [19]

-<p v-else>これまでに参加したプロジェクトはまだ登録されていません</p>
+<p v-else>現在、参加したプロジェクトは登録されていません。</p>
Suggestion importance[1-10]: 6

Why: This suggestion aims to improve consistency in the language used across different parts of the application, which can help in maintaining a uniform user experience.

6

Copy link
Contributor

@Futadaruma Futadaruma left a comment

Choose a reason for hiding this comment

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

プレビューで確認しました
issue の要件を満たしてて良さそうです!
ありがとうございます

Copy link
Collaborator

@Pugma Pugma left a comment

Choose a reason for hiding this comment

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

僕もいいと思います
ありがとう!

@Kaki256 Kaki256 merged commit e53ed46 into main Nov 17, 2024
12 checks passed
@Kaki256 Kaki256 deleted the Pina641/issue205 branch November 17, 2024 12:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ユーザーの情報が入力されていないときの表示を修正
3 participants