? Failure

User tests: Successful: Unsuccessful:

avatar pmorrisarctg
pmorrisarctg
2 Dec 2014
  • Can now install on MS SQL
  • Will create a SQL Server DB if one does not exist
  • Can run the CMS and Admin on SQL Server
avatar pmorrisarctg pmorrisarctg - open - 2 Dec 2014
avatar jissues-bot jissues-bot - change - 2 Dec 2014
Labels Added: ?
avatar pmorrisarctg
pmorrisarctg - comment - 2 Dec 2014

The original pull request that had merge issues due to other developers fixing issues in the same file is here. #3751 That request has merge issues. This one should not.

avatar wilsonge
wilsonge - comment - 3 Dec 2014

Can you deal with the code style issues (several places where you're using spaces for indents instead of tabs plus a small number of other minor issues)

avatar pmorrisarctg
pmorrisarctg - comment - 3 Dec 2014

Absolutely. Will do so and re-commit.

avatar wilsonge
wilsonge - comment - 3 Dec 2014

You're a star! Thanks :)

avatar Bakual
Bakual - comment - 3 Dec 2014

Fixed the codestyle issues and merged into staging. Thanks!

avatar Bakual Bakual - close - 3 Dec 2014
avatar Bakual Bakual - change - 3 Dec 2014
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2014-12-03 07:51:35

Add a Comment

Login with GitHub to post a comment