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

Jump to bottom

Make getDenyHtaccessContent method public #15062

Merged
merged 1 commit into from Oct 28, 2019
Merged

Make getDenyHtaccessContent method public #15062

merged 1 commit into from Oct 28, 2019

Conversation

tsteur
Copy link
Member

@tsteur tsteur commented Oct 28, 2019

@tsteur tsteur added not-in-changelog For issues or pull requests that should not be included in our release changelog on matomo.org. Needs Review PRs that need a code review labels Oct 28, 2019
@tsteur tsteur modified the milestones: 3.12.0, 3.13.0 Oct 28, 2019
sgiehl
sgiehl approved these changes Oct 28, 2019
View reviewed changes
Copy link
Member

@sgiehl sgiehl left a comment

Choose a reason for hiding this comment

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

Guess wouldn't hurt to merge it for 3.12 already

@tsteur tsteur merged commit 97d901c into 3.x-dev Oct 28, 2019
@tsteur tsteur deleted the getdenyhtaccess branch October 28, 2019 17:13
tsteur added a commit that referenced this pull request Oct 30, 2019
@tsteur
Make getAllowHtaccessContent method public #15062
c40d08a 
Similar to #15062 needed for matomo-org/matomo-for-wordpress#50

@sgiehl fyi will merge this one too
@tsteur tsteur mentioned this pull request Oct 30, 2019
Merged
tsteur added a commit that referenced this pull request Oct 30, 2019
@tsteur
Make getAllowHtaccessContent method public #15062 (#15074)
87d7808 
Similar to #15062 needed for matomo-org/matomo-for-wordpress#50

@sgiehl fyi will merge this one too
diosmosis pushed a commit that referenced this pull request Nov 6, 2019
@tsteur @diosmosis
Make getAllowHtaccessContent method public #15062 (#15074)
856b8c8 
Similar to #15062 needed for matomo-org/matomo-for-wordpress#50

@sgiehl fyi will merge this one too
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sgiehl sgiehl sgiehl approved these changes

Assignees
No one assigned
Labels
Needs Review PRs that need a code review not-in-changelog For issues or pull requests that should not be included in our release changelog on matomo.org.
Projects
None yet
Milestone
3.13.0
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@tsteur @sgiehl