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

SQL completed #21

Merged
merged 3 commits into from
Jun 29, 2016
Merged

SQL completed #21

merged 3 commits into from
Jun 29, 2016

Conversation

staslyakhov
Copy link
Collaborator

Same ATM functions but works with a database
(SQL)

@otakaran
Copy link
Member

I'm getting a lot of errors while just trying to deposit / withdraw..
Also, whenever I enter a nonexistent card number the program exits with code 0, this is not ideal.

@otakaran
Copy link
Member

That being said I'll merge this seeing as you're probably the only one who will be working on this

@otakaran otakaran merged commit 9357c74 into master Jun 29, 2016
@staslyakhov
Copy link
Collaborator Author

That's weird... it works for me...

@otakaran
Copy link
Member

screen shot 2016-06-29 at 2 55 12 pm

@staslyakhov
Copy link
Collaborator Author

screenshot 5

@otakaran
Copy link
Member

I suspect there is some code that is windows proprietary and / or there is a dependency on some libraries?

@staslyakhov
Copy link
Collaborator Author

What are you running it with?
It could also be having cross platform issues but that would be weird

@otakaran
Copy link
Member

I'm running it straight through terminal in the python console. I'm on version 3.6.

@staslyakhov
Copy link
Collaborator Author

i have 3.5
btw can you delete the line that's causing the error and try again? its not even needed

@otakaran
Copy link
Member

otakaran commented Jun 29, 2016

Removing line 20 fixed the issue for me.

@staslyakhov
Copy link
Collaborator Author

So it works now?
which one is that?

@otakaran
Copy link
Member

Sorry typo, line 20.

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