Skip to content

Logs out user from Chatrix on non-logged in page load#47

Merged
ashfame merged 3 commits into
mainfrom
wp-chatrix-logout
Aug 26, 2022
Merged

Logs out user from Chatrix on non-logged in page load#47
ashfame merged 3 commits into
mainfrom
wp-chatrix-logout

Conversation

@ashfame

@ashfame ashfame commented Aug 24, 2022

Copy link
Copy Markdown
Member

This PR injects inline JS code when a user is not logged in on any page i.e. theme page or wp-login.php page where the user may land after logging out of WordPress. The injected code finds any chatrix sessions and invalidates those sessions.

Fixes #5

@ashfame ashfame requested a review from psrpinto August 24, 2022 08:43
@ashfame ashfame self-assigned this Aug 24, 2022
Comment thread wordpress/chatrix/src/plugin.php
@ashfame ashfame force-pushed the wp-chatrix-logout branch from 193c499 to 032c7db Compare August 25, 2022 11:09
@ashfame ashfame force-pushed the wp-chatrix-logout branch from 801ec26 to b443c72 Compare August 25, 2022 15:33
Comment thread wordpress/chatrix/src/plugin.php Outdated

@psrpinto psrpinto left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

🚀

@ashfame ashfame merged commit ffa8a84 into main Aug 26, 2022
@ashfame ashfame deleted the wp-chatrix-logout branch August 26, 2022 11:27
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.

How will logout/switching WordPress.org accounts be handled?

2 participants