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

flarum info command additions #2991

Merged
merged 4 commits into from Jul 31, 2021
Merged

flarum info command additions #2991

merged 4 commits into from Jul 31, 2021

Conversation

luceos
Copy link
Member

@luceos luceos commented Jul 30, 2021

This PR adds a couple of new details to the info command:

  • MySQL version, so that we can spot mysql issues faster
  • Queue driver, so that we can understand issues where email isn't sent when the queue worker isn't running
  • Mail driver, so that we can understand issues where email isn't sent when set to log, null or something else

I've also added a newline above the debug warning and I've made debug string "ON" red.

image

@luceos luceos requested review from davwheat and SychO9 July 30, 2021 09:17
@luceos luceos requested a review from SychO9 July 30, 2021 16:03
@SychO9 SychO9 added this to the 1.0.5 milestone Jul 30, 2021
@luceos luceos merged commit 9490b3d into master Jul 31, 2021
@luceos luceos deleted the dk/info-command-additions branch July 31, 2021 10:34
@SychO9 SychO9 modified the milestones: 1.0.5, 1.1 Aug 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tankerkiller125 tankerkiller125 tankerkiller125 approved these changes

@SychO9 SychO9 SychO9 approved these changes

@davwheat davwheat Awaiting requested review from davwheat

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
1.1
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@luceos @tankerkiller125 @SychO9