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

月額変更届のデータレコードを作成 #53

Merged
merged 2 commits into from
Nov 8, 2019

Conversation

wata727
Copy link
Contributor

@wata727 wata727 commented Nov 5, 2019

参照 https://www.nenkin.go.jp/denshibenri/setsumei/20180305.files/01.pdf

P169にある仕様に沿って、月額変更届のデータレコードを作成しました。

@wata727 wata727 requested a review from a team as a code owner November 5, 2019 11:11
@ghost ghost requested review from mserizawa and tknzk and removed request for a team November 5, 2019 11:11
@wata727 wata727 force-pushed the geppen branch 2 times, most recently from b95ef50 to 95e0d3b Compare November 5, 2019 13:57
@@ -186,7 +186,7 @@ end
| 資格喪失届データレコード | Kirico::DataRecord22017041 | △ |
| 住所変更届データレコード | Kirico::DataRecord22187041 | ◯ |
| 算定基礎届データレコード | Kirico::DataRecord2225700 | ◯ |
| 月額変更届データレコード | Kirico::DataRecord22217041 | |
| 月額変更届データレコード | Kirico::DataRecord2221700 | |
Copy link
Member

Choose a reason for hiding this comment

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

🎉

area_code_of_basic_pension_number: 基礎年金番号 課所符号
serial_number_of_basic_pension_number: 基礎年金番号 一連番号
seventy_years_and_over: 70歳以上被用者
work_in_multiple_company: 二以上勤務
Copy link
Member

Choose a reason for hiding this comment

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

これ用語として「二以上事業所勤務」だったりしませんかね・・・?

Copy link
Contributor Author

@wata727 wata727 Nov 6, 2019

Choose a reason for hiding this comment

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

🤔

仕様書上はこれに名前が付いてなかったので、他の算定基礎や賞与支払届の命名にあわせています。まとめて変えた方が良いですかねー

work_in_multiple_company: 二以上勤務

Copy link
Member

Choose a reason for hiding this comment

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

なるほど〜了解しました!
一旦このプルリクでは OK です 🙆

Copy link
Member

@mserizawa mserizawa left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@wata727
Copy link
Contributor Author

wata727 commented Nov 6, 2019

「70歳以上被用者届のみ提出」に「1」を設定した場合に、被保険者整理番号を省略できる仕様を見逃していたので、追加対応しました 730fb8b

@tknzk こちらレビューお願いします

Copy link
Contributor

@tknzk tknzk left a comment

Choose a reason for hiding this comment

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

👍

attribute :retroacted_payment_at, Date
attribute :retroacted_payment, Integer
[3, 2, 1].each do |month|
attribute "before_#{month}_month_days"
Copy link
Member

@wakasa51 wakasa51 Nov 7, 2019

Choose a reason for hiding this comment

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

クラスを指定していなさそうですけど、この場合はStringになりますかね?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

算定基礎の実装スタイルに合わせているのですが、未指定の場合は実際の値から推論されるようですね
一応明確に書いておきましょう

Copy link
Member

@wakasa51 wakasa51 left a comment

Choose a reason for hiding this comment

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

良さそうです!!
お疲れ様です!

@wata727 wata727 merged commit 0c1651d into kufu:master Nov 8, 2019
@wata727 wata727 deleted the geppen branch November 8, 2019 01:34
@wata727 wata727 mentioned this pull request Nov 8, 2019
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