Skip to content

[4.0] installation with postgresql#24747

Merged
wilsonge merged 1 commit intojoomla:4.0-devfrom
alikon:patch-122
Apr 29, 2019
Merged

[4.0] installation with postgresql#24747
wilsonge merged 1 commit intojoomla:4.0-devfrom
alikon:patch-122

Conversation

@alikon
Copy link
Contributor

@alikon alikon commented Apr 28, 2019

Pull Request for Issue #24740

Summary of Changes

fix sql for postresql

Testing Instructions

install with postgresql

Expected result

able to install

Actual result

unable to install

@twister65
Copy link
Contributor

I have tested this item ✅ successfully on dc07eca

Thanks @alikon !


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/24747.

@alikon
Copy link
Contributor Author

alikon commented Apr 28, 2019

@twister65 thanks to you for your work on joomla-framework/database#151

@richard67
Copy link
Member

I have tested this item ✅ successfully on dc07eca


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/24747.

@alikon
Copy link
Contributor Author

alikon commented Apr 28, 2019

thanks for testing
RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/24747.

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Apr 28, 2019
@wilsonge wilsonge merged commit a0ffa2c into joomla:4.0-dev Apr 29, 2019
@wilsonge
Copy link
Contributor

Thanks!

@wilsonge wilsonge added this to the Joomla 4.0 milestone Apr 29, 2019
@joomla-cms-bot joomla-cms-bot added PR-4.0-dev and removed RTC This Pull Request is Ready To Commit labels Apr 29, 2019
@alikon alikon deleted the patch-122 branch April 29, 2019 10:58
@alikon
Copy link
Contributor Author

alikon commented Apr 29, 2019

@wilsonge please keep an eye on #24319 and #24148 are both related to postgresql

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.

5 participants