Deleted unnecessary checks if certain fields were None

[cjappl]

Closes #11
#7
#8

[cjappl]

I think #9 is the much more correct way of doing this, and should be preferred if you have the chance to merge that.

[jazzmoe]

I encountered the same issues as in #8 and would appreaciate a merge if @cjappl 's fix solves the issue

Also I am pretty new to coding in python - can I somehow use your fix as a module @cjappl without waiting for a merge in this project?

[cjappl]

As a temporary measure, you’re welcome to work off the main branch of my fork. I have recently fixed a few more issues where optional fields would throw. Based on the state of this main fork, I think the maintainer is a little MIA

…

On Sat, May 22, 2021 at 7:05 AM, Moritz Hemmerlein ***@***.***> wrote: I encountered the same issues as in [#8](#8) and would appreaciate a merge if ***@***.***(https://github.com/cjappl) 's fix solves the issue — You are receiving this because you were mentioned. Reply to this email directly, [view it on GitHub](#13 (comment)), or [unsubscribe](https://github.com/notifications/unsubscribe-auth/ADMDXY5F3I2WWVI25NAVFPLTO6TX7ANCNFSM436N5LQA).

[jazzmoe]

I'd like to contribute - I actively try to build a useful application linking my different bank accounts to ynab. Can I somehow download your version to my python package folder. How would that work?

[cjappl]

@jazzmoe you can find my fork here

https://github.com/cjappl/ynab-python

You can install directly with

pip install git+https://github.com/cjappl/ynab-python.git

Or, you can clone the repo, navigate to the top level directory and install locally

git clone https://github.com/cjappl/ynab-python
cd ynab-python
pip install -e .

If you have any issues, head over to my fork and report it, we can work on it together!

[ohare93]

Lovely, thank you for this easy fix 👍

[cjappl]

Closing, seems like this repo is dead. RIP