Skip to content

Add sql-custom back#592

Merged
Jason2605 merged 13 commits intoAsYetUntitled:masterfrom
Jason2605:feature/sql-custom
Dec 31, 2019
Merged

Add sql-custom back#592
Jason2605 merged 13 commits intoAsYetUntitled:masterfrom
Jason2605:feature/sql-custom

Conversation

@Jason2605
Copy link
Copy Markdown
Member

@Jason2605 Jason2605 commented Dec 30, 2019

SQL Custom

Changes proposed in this pull request:

  • Change current SQL queries to prepared statements
  • Remove all mres functions
  • Slight formatting updates

Note: This PR was made from #414 so there may be things missing, will leave it in a draft PR until we are more confident we have all the recent changes, and nothing is missing from the old merge.

  • I have tested my changes and corrected any errors found

@Jason2605 Jason2605 added enhancement For pull requests and issues that suggest an improvement to an existing feature. 6.0.0 labels Dec 30, 2019
@Jason2605 Jason2605 self-assigned this Dec 30, 2019
@Jason2605 Jason2605 marked this pull request as ready for review December 30, 2019 22:24
DomT602
DomT602 previously approved these changes Dec 31, 2019
Copy link
Copy Markdown
Member

@DomT602 DomT602 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested with Jason, seems good.

@Jason2605 Jason2605 requested a review from BoGuu December 31, 2019 01:33
@IceEagle132
Copy link
Copy Markdown

Should add in the east side, you already have 3 out of 4 sides. Might as well put them in.

@DomT602 DomT602 requested review from Wackbatt and klmunday December 31, 2019 13:37
@DomT602
Copy link
Copy Markdown
Member

DomT602 commented Dec 31, 2019

Should add in the east side, you already have 3 out of 4 sides. Might as well put them in.

This PR's purpose is to move to SQL Custom, replacing the old SQL Raw, the support for EAST would be done through a seperate PR otherwise it can get very hard to review and track changes. Once this PR is merged, feel free to start a PR to finish EAST support!

Copy link
Copy Markdown
Member

@DomT602 DomT602 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

With recent fixes to gang disbandment, tested and works.

@Jason2605 Jason2605 merged commit 84c4f7c into AsYetUntitled:master Dec 31, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

6.0.0 enhancement For pull requests and issues that suggest an improvement to an existing feature.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants