Skip to content

Neos 9.0 compatibility#49

Merged
Sebobo merged 4 commits intoFlowpack:mainfrom
kdambekalns:task/neos-9.0-compatibility
Apr 15, 2025
Merged

Neos 9.0 compatibility#49
Sebobo merged 4 commits intoFlowpack:mainfrom
kdambekalns:task/neos-9.0-compatibility

Conversation

@kdambekalns
Copy link
Copy Markdown
Member

No description provided.

@kdambekalns kdambekalns self-assigned this Apr 3, 2025
@kdambekalns kdambekalns changed the title Task/neos 9.0 compatibility Neos 9.0 compatibility Apr 3, 2025
@dlubitz dlubitz mentioned this pull request Apr 11, 2025
@Sebobo
Copy link
Copy Markdown
Member

Sebobo commented Apr 15, 2025

@kdambekalns is there still something missing in this PR? Would review if you un-draft it.

@dlubitz
Copy link
Copy Markdown
Contributor

dlubitz commented Apr 15, 2025

We have now two PRs for Neos 9 compatibility.
See: #48

@kdambekalns
Copy link
Copy Markdown
Member Author

I don't really know yet. This is the result of what I needed to do at minimum, but I did not find the time to progress with the upgrade I did this for. 🙈

@Sebobo Sebobo marked this pull request as ready for review April 15, 2025 10:24
@Sebobo
Copy link
Copy Markdown
Member

Sebobo commented Apr 15, 2025

I merged the other one, but this one has a few additional cleanups. I can take a look later to finish this up.

@Sebobo Sebobo requested review from Sebobo and dlubitz April 15, 2025 14:58
@Sebobo
Copy link
Copy Markdown
Member

Sebobo commented Apr 15, 2025

Cleaned up the PR after the merge and conflict resolution. Looks good and throws no errors. Haven't tested an actual connection yet.

Copy link
Copy Markdown
Contributor

@dlubitz dlubitz left a comment

Choose a reason for hiding this comment

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

Thank you! Looks good!

@Sebobo Sebobo merged commit 41d7152 into Flowpack:main Apr 15, 2025
@kdambekalns kdambekalns deleted the task/neos-9.0-compatibility branch June 25, 2025 17:57
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.

3 participants