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

Fix for #13761 #13835

Merged
merged 1 commit into from Mar 26, 2018
Merged

Fix for #13761 #13835

merged 1 commit into from Mar 26, 2018

Conversation

bezumkin
Copy link
Contributor

What does it do?

This PR will fix "Who`s online" on servers with MySQL 5.7 with enabled "sql_mode=only_full_group_by"

Why is it needed?

Because this widget produces error in manager log and not working.

Related issue(s)/PR(s)

#13730 #13761

PR was recreated, because I can`t update the old one.

@opengeek opengeek merged commit 591c0d5 into modxcms:2.6.x Mar 26, 2018
@opengeek opengeek added this to the v2.6.2 milestone Mar 26, 2018
@opengeek opengeek added type-frontend Issues related to UI/UX issues, mostly about styles and frontend implementations on JavaScript. bug The issue in the code or project, which should be addressed. area-core labels Mar 26, 2018
@bezumkin bezumkin deleted the 2x-whois-online branch March 27, 2018 06:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-core bug The issue in the code or project, which should be addressed. type-frontend Issues related to UI/UX issues, mostly about styles and frontend implementations on JavaScript.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants