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

feat/申請詳細ページのファイル周り #296

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

anko9801
Copy link
Contributor

@anko9801 anko9801 commented Sep 18, 2024

User description

fix: #169


PR Type

enhancement


Description

  • RequestContent.vueにファイル表示とアップロード機能を追加しました。これにより、ユーザーはファイルを表示およびアップロードできるようになりました。
  • RequestDetailPage.vueRequestLogsコンポーネントのスタイルを調整し、レイアウトの見た目を改善しました。

Changes walkthrough 📝

Relevant files
Enhancement
RequestContent.vue
ファイル表示とアップロード機能の追加                                                                             

src/components/requestDetail/RequestContent.vue

  • ファイル表示用のデータ構造を追加
  • ファイルアップロードのハンドラーを追加
  • ファイル表示のUIコンポーネントを追加
  • ファイルアップロードボタンを追加
  • +79/-3   
    RequestDetailPage.vue
    レイアウトのスタイル調整                                                                                         

    src/pages/RequestDetailPage.vue

    • RequestLogsコンポーネントのスタイルを調整
    +1/-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: 3 🔵🔵🔵⚪⚪
    🧪 No relevant tests
    🔒 No security concerns identified
    ⚡ Key issues to review

    重複コード
    ファイルの配列がハードコードされており、同じファイルが複数回繰り返されています。これにより、メンテナンス性が低下しています。配列の要素を一意にするか、データのソースを動的にすることを検討してください。

    パフォーマンス問題
    大量のファイルデータをクライアントサイドでハードコードしているため、ページの読み込み速度に影響を与える可能性があります。ファイルデータをサーバーから非同期に取得する方法を検討してください。

    @anko9801 anko9801 requested a review from reiroop December 21, 2024 11:27
    @anko9801
    Copy link
    Contributor Author

    anko9801 commented Dec 21, 2024

    TODO: この API 設計だとファイル名とファイルタイプが分からないのでバックエンド側にお願いする。

    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.

    申請詳細ページのファイル周り
    1 participant