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.

Sign up for GitHub

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

Jump to bottom

Remove payee lookup #50

Closed
scottrobertson opened this issue Feb 1, 2018 · 3 comments
Closed

Remove payee lookup #50

scottrobertson opened this issue Feb 1, 2018 · 3 comments
Labels
blocked enhancement
Milestone
YNAB API Rewrite

Comments

@scottrobertson
Copy link
Collaborator

Once YNAB support passing in payee_name, we can remove all our logic around payee lookups. This should happen mid next week.
@scottrobertson scottrobertson added this to the YNAB API Rewrite milestone Feb 1, 2018
@scottrobertson
Copy link
Collaborator Author

Right now this is kind of messed up as if the payee does not exist, it just creates the transaction with no payee at all.

@scottrobertson
Copy link
Collaborator Author

This has been removed now. All we need to do now is pass payee_name to the create endpoint and it handles the creation if needed.

@scottrobertson
Copy link
Collaborator Author

Keeping the payee_id lookup around for duplicate detection for now until the import_id stuff is implemented.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
blocked enhancement
Projects
None yet
Milestone
YNAB API Rewrite
Development

No branches or pull requests
1 participant
@scottrobertson