Skip to content

Conversation

@richard67
Copy link
Member

@richard67 richard67 commented May 13, 2021

Pull Request for #33550 (comment) .

Summary of Changes

See the title and the referred comment.

Testing Instructions

  1. Code review.
  2. Check that all system tests have passed in drone, which means there were no SQL errors on installation with MySQL and PostgreSQL databases.

Or alternatively, make a new installation and check if table #__csp exists in database.

Actual result BEFORE applying this Pull Request

Table #__csp exists in database after a new installation.

Expected result AFTER applying this Pull Request

Table #__csp doesn't exist in database after a new installation.

Additional information

For updates from 3.10 or from previous 4.0 Beta versions, everything is ok already, see #33820 .

Documentation Changes Required

No.

@joomdonation
Copy link
Contributor

I have tested this item ✅ successfully on 242da63


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

1 similar comment
@Quy
Copy link
Contributor

Quy commented May 13, 2021

I have tested this item ✅ successfully on 242da63


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

@joomla-cms-bot joomla-cms-bot removed this from the Joomla 4.0 milestone May 13, 2021
@Quy
Copy link
Contributor

Quy commented May 13, 2021

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label May 13, 2021
@Quy Quy added this to the Joomla 4.0 milestone May 13, 2021
@chmst chmst merged commit fe7e196 into joomla:4.0-dev May 13, 2021
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label May 13, 2021
@richard67 richard67 deleted the 4.0-dev-complete-csp-removal branch May 13, 2021 21:31
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