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

iOS NFCReader切り出し対応(APDUコマンド切り出し) #1

Open
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

mizuno-bks
Copy link
Collaborator

・マインバーカードの電子証明書読み取り処理の追加
・APDUコマンドでInfo.plistの値を使用するように修正

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@Sheile
Copy link

Sheile commented Sep 9, 2023

@mizuno-bks
修正コミットが別に入ってしまっていますが、PRとして出す以上きれいなコミット履歴にしておく必要があります。
本来はgit commit --fixupなどで既存コミットをするのが良いのですが、Gitに不慣れだと難しいと思うので全部まとめて1コミットにするぐらいでしょうか。

また、コミットメッセージもコミットと合っていないように見えます。
例: マイナンバーカード読み取り処理を追加
マイナンバーカードの読み取り処理自体は元々あって、今回追加したのは「公的個人認証APを用いた証明書の読み取り・署名」ぐらいでしょうか。

@Sheile
Copy link

Sheile commented Sep 25, 2023

残課題についてはIssueに記載して別途対応としています。
Fork元にPRを出す際に一連の修正を入れたブランチがあった方が良いため、マージ後もこちらのブランチは維持した方が良いと思います。

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.

2 participants