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)作成機能の追加@協賛活動一覧ページ #712

Merged
merged 14 commits into from
Mar 29, 2024

Conversation

TkymHrt
Copy link
Collaborator

@TkymHrt TkymHrt commented Mar 20, 2024

対応Issue

resolve #704

概要

協賛活動一覧ページにおいて、請求書(PDF)作成機能を追加する。

画面スクリーンショット等

FireShot Capture 002 - 協賛活動一覧 - localhost

2024-03-21-テスト企業-請求書

FireShot Capture 006 - 協賛活動一覧 - localhost

テスト項目

  • http://localhost:3000/sponsoractivities へアクセスし、登録された項目をクリックして選択すると「請求書作成」ボタンが表示される。
  • 「請求書作成」ボタンを押すことで、請求書のPDFファイルがダウンロードされることを確認する。
  • 登録されている内容がPDFファイルに反映されていることを確認する。

備考

請求書 テンプレート

@TkymHrt TkymHrt changed the title 請求書(PDF)作成機能の追加@ 請求書(PDF)作成機能の追加@協賛活動一覧ページ Mar 20, 2024
@TkymHrt TkymHrt self-assigned this Mar 20, 2024
@hikahana
Copy link
Collaborator

請求書の概要部分に協賛スタイルのオプションを追記(モノクロ、カラー等)

請求期限に関してはフォームを新たに作成して、期限を入力させる。
参考にのぶのPRを貼る
#709

@hikahana
Copy link
Collaborator

備考はモーダルの備考ではない。
新たに入力させる

Copy link
Collaborator

@hikahana hikahana left a comment

Choose a reason for hiding this comment

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

動作okでした!
半分くらい個人的に気になったところだけどコメントつけたので時間ある時にでも目を通してください

<View>
<View style={styles.marginButtom}>
<Text>技大祭実行委員</Text>
<Text>〒940-2137</Text>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
<Text>〒940-2137</Text>
<Text>〒940-2188</Text>

技大のHP的に郵便番号これらしい
image

Copy link
Collaborator

Choose a reason for hiding this comment

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

940-2137も上富岡町の郵便番号で合ってるよ、24がどっちなのかは確認した方がいいかも

<View style={styles.detailItem}>
<View style={styles.textVertical}>
<Text style={[styles.text_M, styles.underLine]}>
{sponsorActivitiesViewItem.sponsor.name} <Text style={styles.text_S}>御中</Text>
Copy link
Collaborator

Choose a reason for hiding this comment

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

御中の位置、下線の横にあってもいいかも?
それか少し余白置いて、文字サイズ企業名と同じ大きさにするかの方がよさげ

ちょっと気になっただけなので修正するかはお任せします

Comment on lines 217 to 218
<Text style={[styles.commonTableCol, styles.tableColHeader_S]}></Text>
<Text style={[styles.commonTableCol, styles.tableColHeader_L]}></Text>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
<Text style={[styles.commonTableCol, styles.tableColHeader_S]}></Text>
<Text style={[styles.commonTableCol, styles.tableColHeader_L]}></Text>
<Text style={[styles.commonTableCol, styles.tableColHeader_S]} />
<Text style={[styles.commonTableCol, styles.tableColHeader_L]} />

私はこっちのほうが好き
直すかはお任せします

</Document>
);
const asPdf = pdf(<MyDocument />);
await asPdf.toBlob();
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
await asPdf.toBlob();

多分なくても動く

);
}}
>
請求書ダウンロード
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
請求書ダウンロード
ダウンロード

請求書作成クリックして、請求書ダウンロードがでてくるのが請求書の圧を感じた

Copy link
Collaborator

@Kubosaka Kubosaka left a comment

Choose a reason for hiding this comment

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

2点お願いします。

  1. ダウンロードボタンを押したら、モーダルは消えた方がいいかもですね。
  2. コンフリクト解消をお願いできますか?
    請求書と領収書作成ボタンが並ぶ感じで大丈夫です。

@TkymHrt TkymHrt requested a review from hikahana March 28, 2024 06:42
@TkymHrt TkymHrt requested a review from Kubosaka March 28, 2024 08:11
Copy link
Collaborator

@Kubosaka Kubosaka left a comment

Choose a reason for hiding this comment

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

LGTM

@Kubosaka Kubosaka merged commit 286f3ac into develop Mar 29, 2024
2 checks passed
@Kubosaka Kubosaka deleted the feat/yama/704-create-invoice-as-pdf branch March 29, 2024 08:40
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.

協賛金の請求書をpdf出力する
3 participants