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

Fix: access filter + anonymous yes/no #9650

Merged
merged 1 commit into from Mar 7, 2024

Conversation

Ruud68
Copy link
Contributor

@Ruud68 Ruud68 commented Mar 7, 2024

Pull Request for Issue #9649

Summary of Changes

in category view:

  • fixed accesslevel filter
  • renames anonymous filter options to yes/no

Testing Instructions

@rich20

@rich20
Copy link
Member

rich20 commented Mar 7, 2024

Yes, it now works in exactly the same way as in the category filter, the parent category must be selected. This is also how it works with the other filters.
The only filter that doesn't work at all now is the Select Access filter.

@Ruud68
Copy link
Contributor Author

Ruud68 commented Mar 7, 2024

@rich20 the access filter should be fixed with this PR?

@rich20
Copy link
Member

rich20 commented Mar 7, 2024

Addendum:
The text Yes/No is now correct. Can be merged.

@rich20
Copy link
Member

rich20 commented Mar 7, 2024

@rich20 the access filter should be fixed with this PR?v

No, it was just a general summary. Sorry!

The text error in the Anonymous filter has been fixed, so this PR can be merged.

@xillibit xillibit added this to the 6.3 milestone Mar 7, 2024
@xillibit xillibit merged commit 55b35c7 into Kunena:K6.3 Mar 7, 2024
4 checks passed
@Ruud68 Ruud68 deleted the categoryfilterfix branch March 8, 2024 07:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants