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

(WIP) CRM-16421 CRM-17633 Changes to support WP in it's own directory R… #9106

Closed
wants to merge 5 commits into from

Conversation

kcristiano
Copy link
Member

…eplaces PR#8536

@kcristiano
Copy link
Member Author

test this please

@litespeedmarc litespeedmarc changed the title CRM-16421 CRM-17633 WIP Changes to support WP in it's own directory R… (WIP) CRM-16421 CRM-17633 Changes to support WP in it's own directory R… Sep 27, 2016
@litespeedmarc litespeedmarc added needs-work-not-review-ready Submitter required to take action, will be closed after 1-2 weeks in this state and removed needs-work-not-review-ready Submitter required to take action, will be closed after 1-2 weeks in this state labels Sep 27, 2016
@seamuslee001
Copy link
Contributor

Jenkins re test this please

@seamuslee001
Copy link
Contributor

@kcristiano Kevin where is this at now?

@kcristiano
Copy link
Member Author

@seamuslee001 this has been tested along with civicrm/civicrm-wordpress#105. However, the installer needs to be updated to get around manual steps detailed here
https://wiki.civicrm.org/confluence/display/CRM/WordPress+installed+in+its+own+directory+issues in the Next Steps section.

4.7.13 is ambitious (unless someone at Edale wants to work on this) but by 4.7.14 this should be ready.

@kcristiano
Copy link
Member Author

jenkins test this please

@kcristiano
Copy link
Member Author

I've tested this and civicrm/civicrm-wordpress#105 against current master. PR still works as advertised. I'd like to get this on a release track so we can thoroughly test

…Allow for civicrn.settings.extra.php for WP. Create template for civicrm.settings.extra.php. Update install/civicrm.php for all needed params for different common install scenarios. Fix linting issues in civicrm.php
@kcristiano
Copy link
Member Author

jenkins test this please

@kcristiano
Copy link
Member Author

Rebased and replaced this PR with #10214

@kcristiano kcristiano closed this Apr 22, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-work-not-review-ready Submitter required to take action, will be closed after 1-2 weeks in this state
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants