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

卒業証書のPDFファイルをアップロードできる機能を追加 #8190

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

Conversation

hagiya0121
Copy link
Contributor

@hagiya0121 hagiya0121 commented Nov 13, 2024

Issue

概要

管理者のユーザー登録情報変更画面で卒業証書のPDFファイルをアップロード及び削除できる機能を追加しました。

変更確認方法

  1. feature/add-diploma-pdf-uploadをローカルに取り込む
    1. git fetch origin pull/8190/head:feature/add-diploma-pdf-upload
    2. git checkout feature/add-diploma-pdf-upload
  2. foreman start -f Procfile.dev でローカルサーバーを立ち上げ
  3. komagata(管理者)でログイン
  4. ユーザー登録情報変更画面にアクセス
  5. 卒業証書PDF項目でtest/fixtures/files/users/diplomas/diploma.pdfをアップロードして更新する
  6. ユーザー登録情報変更画面にアクセス
  7. 卒業証書PDF項目にdiploma.pdfと表示され、PDFを確認を押してPDFが表示されることを確認する
  8. 卒業証書PDF項目で削除ボタンを押して更新する
  9. ユーザー登録情報変更画面にアクセス
  10. 卒業証書PDF項目でdiploma.pdfという表示が消えていることを確認

Screenshot

変更前

image

変更後

image

@hagiya0121
Copy link
Contributor Author

@machida
お疲れ様です。デザインをお願いしたいです🙏

@machida
Copy link
Member

machida commented Nov 14, 2024

@hagiya0121 了解です!!

@machida
Copy link
Member

machida commented Nov 18, 2024

@hagiya0121 お待たせしました!!デザイン入れましたのでご確認お願いします。OKでしたらレビューに進めてくださいー

@machida machida removed their assignment Nov 18, 2024
@hagiya0121
Copy link
Contributor Author

@ham-cap
お疲れ様です。
こちらのPRのレビューをお願いしたいです🙏
ご都合が悪いときは遠慮なくおっしゃってください🙇

@hagiya0121 hagiya0121 marked this pull request as ready for review November 23, 2024 22:45
@ham-cap
Copy link
Contributor

ham-cap commented Nov 28, 2024

@hagiya0121
返信が大変遅くなってしまい申し訳ありません🙇‍♂️
先日チーム開発のプラクティスを修了してしまったところなので、別の方にご依頼いただけますでしょうか🙏
恐縮ですがよろしくお願いいたします🙏

@hagiya0121
Copy link
Contributor Author

hagiya0121 commented Nov 28, 2024

@ham-cap
分かりました、ご連絡いただきありがとうございます🙇

@hagiya0121 hagiya0121 requested review from mousu-a and removed request for ham-cap November 28, 2024 07:38
@hagiya0121
Copy link
Contributor Author

@mousu-a
お疲れ様です。
こちらのPRのレビューをお願いしたいです🙏
ご都合が悪いときは遠慮なくおっしゃってください🙇

@mousu-a
Copy link
Contributor

mousu-a commented Nov 28, 2024

@hagiya0121
レビューありがとうございます!
ぜひ引き受けさせていただきます〜😄
一週間ほどかかりそうなスケジュール感です🙇‍♂️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants