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

リンクが指定されていないときに表示されないようにした #198

Merged
merged 1 commit into from
Jul 16, 2024

Conversation

sh0go07
Copy link
Contributor

@sh0go07 sh0go07 commented Jul 10, 2024

User description

close #190


PR Type

enhancement


Description

  • external-link コンポーネントに v-if ディレクティブを追加し、contestTeamDetail.link が存在する場合のみリンクを表示するように変更。

Changes walkthrough 📝

Relevant files
Enhancement
ContestTeamPage.vue
リンクが指定されていない場合の表示制御を追加                                                                     

src/pages/ContestTeamPage.vue

  • external-link コンポーネントに v-if ディレクティブを追加
  • contestTeamDetail.link が存在する場合のみリンクを表示
  • +4/-1     

    💡 PR-Agent usage:
    Comment /help on the PR to get a list of all available PR-Agent tools and their descriptions

    Copy link

    PR Reviewer Guide 🔍

    ⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
    🧪 No relevant tests
    🔒 No security concerns identified
    ⚡ No key issues to review

    Copy link

    PR Code Suggestions ✨

    CategorySuggestion                                                                                                                                    Score
    Enhancement
    リンクが無効な場合も考慮して、表示条件を強化する。


    v-ifディレクティブを使用して、contestTeamDetail.linkが存在する場合のみコンポーネントを表示するように修正しましたが、リンクが無効(空文字やnull)の場合にも考慮する必要があります。contestTeamDetail.linkの値をチェックする条件を強化して、空文字やnullでないことを確認してください。

    src/pages/ContestTeamPage.vue [39-41]

     <external-link
    -  v-if="contestTeamDetail.link"
    -  :href="contestTeamDetail.link"
    +  v-if="contestTeamDetail.link && contestTeamDetail.link.trim() !== ''"
    +  :href="contestTeamLink.link"
     >
     
    Suggestion importance[1-10]: 8

    Why: The suggestion correctly identifies a potential issue where the link might be an empty string or null, and provides a robust condition to handle such cases, improving the reliability of the component.

    8

    @ogu-kazemiya ogu-kazemiya self-requested a review July 12, 2024 13:04
    Copy link
    Contributor

    @ogu-kazemiya ogu-kazemiya left a comment

    Choose a reason for hiding this comment

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

    よさそうです

    @sh0go07 sh0go07 merged commit 25d82c0 into master Jul 16, 2024
    9 checks passed
    @sh0go07 sh0go07 deleted the contest_url branch July 16, 2024 08:50
    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.

    コンテストのURLが指定されていないときは導線を消す
    2 participants