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

Fixing the incorrect table name in server.lua and updating the README with the solution for the properties.sql import issue. #122

Merged
merged 3 commits into from
Aug 27, 2023

Conversation

Ruby-LK
Copy link
Contributor

@Ruby-LK Ruby-LK commented Aug 16, 2023

Overview

This PR should fix the issue with sql query which calls the wrong table in the db and I have added some details about the solution for the properties.sql import issue : Foreign key constraint is incorrectly formed!

Details

As I described in the overview there was a problem in the server.lua (line 115) which calls wrong table : apartments, instead of the properties table.
And I have added an additional part to the readme which tells how to fix that "Foreign key constraint is incorrectly formed" error.

UI Changes / Functionality

  • No UI changes

Testing Steps

  • Just download the release and config it as in the readme.

  • Create a new character and see the server console.

  • Make sure to use a completely wiped database. (without any previous player data.)

  • Did you test the changes you made? = Yes

  • Did you test core functionality of the script to ensure your changes do not regress other areas? = Yes

  • Did you test your changes in multiplayer to ensure it works correctly on all clients? = Yes

@MonkeyWhisper
Copy link
Member

Hi @Ruby-LK

Please fix conflicts, will merge after, thanks

@MonkeyWhisper MonkeyWhisper added ready for merge Reviewed and can be merged to main and removed needs testing labels Aug 22, 2023
@Ruby-LK
Copy link
Contributor Author

Ruby-LK commented Aug 24, 2023

Hey, @MonkeyWhisper

The conflicts that were present in the readme have been resolved.
Hope everything is good now.

Thanks.

@MonkeyWhisper MonkeyWhisper merged commit adcee6e into Project-Sloth:main Aug 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready for merge Reviewed and can be merged to main
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants