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

CRM-19370 - (NFC) Fix post upgrade message for 4.7.11 #9042

Merged
merged 2 commits into from
Sep 15, 2016

Conversation

seamuslee001
Copy link
Contributor

@seamuslee001 seamuslee001 commented Sep 15, 2016

@@ -103,7 +103,7 @@ public function setPostUpgradeMessage(&$postUpgradeMessage, $rev) {
// $postUpgradeMessage .= '<br /><br />' . ts("WARNING: For increased security, profile submissions embedded in remote sites are no longer allowed to create or edit data by default. If you need to allow users to submit profiles from external sites, you can restore this at Administer > System Settings > Misc (Undelete, PDFs, Limits, Logging, Captcha, etc.) > 'Accept profile submissions from external sites'");
//}
if ($rev == '4.7.11') {
$postUpgradeMessage .= '<br /><br />' . ts("By default, CiviCRM now disables the ability to import directly fro SQL. To use this feature, you must explicitly grant permission 'import SQL datasource'.");
Copy link
Contributor

Choose a reason for hiding this comment

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

Shouldn't this be from?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ah yes thanks

@yashodha
Copy link
Contributor

@seamuslee001 : wanna file an issue with JIRA as well?

@yashodha yashodha changed the title (NFC) Fix post upgrade message for 4.7.11 CRM-19370 - (NFC) Fix post upgrade message for 4.7.11 Sep 15, 2016
@yashodha yashodha merged commit 05a2160 into civicrm:master Sep 15, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants