Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Always register the admin filters #215

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

akirk
Copy link
Member

@akirk akirk commented Apr 14, 2023

On installs where the wp-admin is not in /, the settings screen is empty (see https://wpgp.kirk.at/?lang=de&plugin=chatrix). This is because we do some premature optimization to not register the filters.

Screenshot 2023-04-14 at 10 03 52

By removing that optimization, the settings load also in the above case.

@akirk akirk requested a review from psrpinto April 14, 2023 09:37
@ashfame
Copy link
Member

ashfame commented Apr 14, 2023

FYI, I do see the admin page on v0.7.1 on a WP install under a subdir

Screenshot 2023-04-14 at 2 00 02 PM

I am not sure we should do this to make Chatrix work on "WP in browser via wasm". IMO, there would be other plugins commonly doing this exact sort of thing & right fix would be in "WP in browser via wasm" codebase, no?

@psrpinto psrpinto removed their request for review January 22, 2024 14:44
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