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

[Dev] Using llmsPostsSelect2 on a group of elements is not reliable #2805

Open
4 tasks
brianhogg opened this issue Nov 14, 2024 · 0 comments
Open
4 tasks

[Dev] Using llmsPostsSelect2 on a group of elements is not reliable #2805

brianhogg opened this issue Nov 14, 2024 · 0 comments
Assignees

Comments

@brianhogg
Copy link
Contributor

brianhogg commented Nov 14, 2024

Reproduction Steps

  1. Add multiple select boxes with the llms-posts-select2 class
  2. Add data-post-type on the second one (ie. llms-group)

Expected Behavior

The second dropdown works as expected, fetching posts from the specified post type

Actual Behavior

It uses the post type from the first dropdown since options is shared and llmsSelect2 is initialized on all the dropdowns at the same time.

Testing

Testing a fix for this will just be making sure dropdowns like the bulk enroll "Select Course/Membership" on the Users page, filters on LifterLMS > Reporting, "Enrollments" tab, etc still work as expected.

This issue has be recreated:

  • Locally
  • On a staging site
  • On a production website
  • With only LifterLMS and a default theme

Browser, Device, and Operating System Information

  • Browser name and version
  • Operating System name and version
  • Device name and version (if applicable)
@ideadude ideadude moved this to Awaiting Triage in Development Nov 14, 2024
@ideadude ideadude self-assigned this Nov 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Awaiting Triage
Development

No branches or pull requests

2 participants