bill-vektor
bill-vektor copied to clipboard
【確認待ち】インボイス制度に対応
チケットへのリンク / 変更の理由(元のissueがあればリンクを貼り付ければOK)
https://github.com/vektor-inc/bill-vektor/issues/175
どういう変更をしたか?
- 入力欄上部の消費税に関する項目を削除
- 金額欄には税抜金額を入力してもらうように変更
- 品目毎に消費税率が設定できるように
- 上記に伴って表示画面・エクスポートを変更
実装者の確認事項
実装者はレビュワーに回す前に以下の事を確認してチェックをつけてください。
- [x] 複数の意図の変更 ( 機能の不具合修正 + 別の機能追加など ) を含んでいないか?
- [x] Files changed (変更ファイル)の内容は目視で確認したか?
- [ ] readme.txt に変更内容は書いたか?
- [x] 本当にちゃんと確認をしたか?
プログラムの変更の場合
テストを書かないのは普通ではありません。書けるテストは極力書くようにしてください。
- [ ] 書けそうなテストは書いたか? ⇒ どこのテストが必要なのかよくわからないので保留
変更内容について何を確認したか、どういう方法で確認をしたかなど
- 適当に見積書・請求書を作成
- 上記がインボイス制度に則った表示になっていることを確認
- エクスポートが正常に行われているのを確認
確認URL
ないのでローカル環境でお願いします。
レビュワーに回す前の確認事項
- [x] 実装者はこのテンプレートのチェック項目をちゃんと確認してチェックしたか?
レビュワー確認方法・確認内容など
レビュワーがどういう手順で何を確認して欲しいかを記載してください。
- 適当に見積書・請求書を作成
- 上記がインボイス制度に則った表示になっていることを確認
- エクスポートが正常に行われているのを確認
- 各種インポートが正常に行われるか確認
- 消費税の計算方法が間違っていないか確認
レビュワー向け
レビュワーが確認して変更が反映されていない場合の確認事項
レビューしてみて意図した動作をしない場合は再度ビルドするなど以下の項目を確認してください。
- プルしたか?
- ビルドしたか?
- ビルドしたディレクトリは正しいか(別の開発環境のディレクトリを見ていないか)?
- npm install したか?
- composer install したか?
- キャッシュをクリアして確認したか?