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

Add check in finpack.core.loader:loader for str in date column #53

Open
RackReaver opened this issue Dec 18, 2021 · 0 comments
Open

Add check in finpack.core.loader:loader for str in date column #53

RackReaver opened this issue Dec 18, 2021 · 0 comments
Labels
enhancement New feature or request
Milestone

Comments

@RackReaver
Copy link
Owner

RackReaver commented Dec 18, 2021

I want to catch the error before it is imported using the Account class into the account list. The data brought in from any date column should be an int or float (it is converted to float no matter what) so it will break somewhere in the code if its a str, I want the break point to be in the loader function.

branch suggestion: docs/add-str-check-to-loader
commit suggestion: docs: :memo: #53 - <commit message>

@RackReaver RackReaver added the enhancement New feature or request label Dec 18, 2021
@RackReaver RackReaver added this to the v0.3.0 milestone Dec 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant