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

APIの認証処理を整理 #42

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from
Open

Conversation

liebe-magi
Copy link
Owner

#38 のPRでAPIの認証処理を簡素化していただいたため、より使いやすくするために以下の変更を実施

  • get_access_tokenをZaimAPIのメンバ関数化し、インスタンスの初期化処理時に実行するように変更
  • それに伴い、ZaimAPIの初期化関数の引数をコンシューマID、コンシューマシークレットのみに変更
  • 実運用を考え、READMEでコンシューマID、コンシューマシークレットを環境変数として扱うように変更 (それに伴い、python-dotenvも導入)

破壊的な変更となるので、ある程度意見を聞いてからマージするか検討したいと思います。

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.

1 participant