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

Changed code #19

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Changed code #19

wants to merge 2 commits into from

Conversation

JericoFX
Copy link
Contributor

  1. Added an auto generate SQL tables
  2. Changed how the atm is managed, you can only open an ATM only if you are close.
  3. Deleted the event and added an export

@mafewtm
Copy link
Contributor

mafewtm commented Jul 24, 2024

There should also be a discussion amongst the Project Sloth team whether we want to be generating the SQL or not. I would also make sure that all your variables and whatnot are in english, not spanish or whatever they are in now.

@JericoFX
Copy link
Contributor Author

The auto generate sql is for people that can import an sql in first place so i don't underesrand the problem there, the variables is an easy change.

@mafewtm
Copy link
Contributor

mafewtm commented Jul 24, 2024

I personally think the SQL should be kept how it is. Everyone can import a SQL file so that's not an issue. The rest of the Project Sloth resources have the separate file so I think we should just maintain that standard. I don't even wanna think about all the new and exciting ways people will manage to fuck this up. That's just me though.

Commented the auto sql creator
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