Skip to content

Merge develop into main - #59

Merged
kayjoosten merged 9 commits into
mainfrom
develop
Feb 23, 2026
Merged

Merge develop into main#59
kayjoosten merged 9 commits into
mainfrom
develop

Conversation

@kayjoosten

Copy link
Copy Markdown
Contributor

No description provided.

@pmeulen pmeulen left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Note: Moving SQL to separate files and testing it were commits by me, so I'm reviewing my own work here too ;)

Comment thread CHANGELOG.md Outdated
# Changelog

## 4.4.0
* Replace cache/filesystem-adapter for symfony/cache to support PHP 8.2+

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

That affects sending FCM (Android) push notification messages only, correct?

Please add this to the changelog, that helps testers and library users understand the impact of the change.

Comment thread CHANGELOG.md Outdated
# Changelog

## 4.4.0
* Replace cache/filesystem-adapter for symfony/cache to support PHP 8.2+

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

There is a pull request for this change, please append it like this to the line in the changelog: (#57)

@kayjoosten
kayjoosten merged commit d39bd8c into main Feb 23, 2026
3 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