Skip to content

8.1.2#1132

Merged
CarlSchwan merged 2 commits into
masterfrom
carl/8.1.2
Jun 2, 2026
Merged

8.1.2#1132
CarlSchwan merged 2 commits into
masterfrom
carl/8.1.2

Conversation

@CarlSchwan
Copy link
Copy Markdown
Member

No description provided.

@CarlSchwan CarlSchwan requested a review from blizzz as a code owner June 1, 2026 16:14
Copy link
Copy Markdown
Member

@blizzz blizzz left a comment

Choose a reason for hiding this comment

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

what's the root cause? DI hell?

@CarlSchwan
Copy link
Copy Markdown
Member Author

what's the root cause? DI hell?

the event triggers a listener in Guest which expect the session to be set. But it seems that while we setup the session just before triggering the listener it is only fully set after the redirect. I need to investigate this a bit more, but reverting the commit fixes the issue for now

@CarlSchwan CarlSchwan enabled auto-merge June 1, 2026 18:04
This reverts commit 0890d21.

Signed-off-by: Carl Schwan <carlschwan@kde.org>
Signed-off-by: Carl Schwan <carlschwan@kde.org>
@CarlSchwan CarlSchwan merged commit c5c6a48 into master Jun 2, 2026
57 checks passed
@CarlSchwan CarlSchwan deleted the carl/8.1.2 branch June 2, 2026 07:13
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