Skip to content

Conversation

@GromNaN
Copy link
Member

@GromNaN GromNaN commented Oct 27, 2025

Q A
Branch? 4.x
Bug fix? no
New feature? no
Deprecations? no
Issues After #550
License MIT

The mongo handler type is replaced by mongodb.

@GromNaN GromNaN requested a review from HypeMC October 27, 2025 23:18
Copy link
Member

@HypeMC HypeMC left a comment

Choose a reason for hiding this comment

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

Fabbot failure should be fixed.

->end();

$this->addGelfSection($handlerNode);
$this->addMongoSection($handlerNode);
Copy link
Member

@HypeMC HypeMC Oct 27, 2025

Choose a reason for hiding this comment

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

The comments on the top should be removed as well.

Copy link
Member

Choose a reason for hiding this comment

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

Also from the XSD.

Copy link
Member Author

Choose a reason for hiding this comment

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

Thank you for catching what I had missed. It 's fixed.

Copy link
Member

Choose a reason for hiding this comment

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

Just noticed, the comments part is still there.

@GromNaN GromNaN force-pushed the remove-mongo branch 2 times, most recently from d15d337 to 0d96f88 Compare October 27, 2025 23:29
}
break;

case 'elasticsearch':
Copy link
Member Author

Choose a reason for hiding this comment

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

@GromNaN GromNaN merged commit e0cd1da into symfony:4.x Oct 28, 2025
8 checks passed
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.

2 participants