search

jameswennmacher / NotificationPortlet

Apache License 2.0
0 stars 0 forks source link

Checking show in person displays all jobs regardless of other filter criteria #10

Closed jameswennmacher closed 10 years ago

jameswennmacher commented 10 years ago

Scenario 1: refresh page check academic. Properly shows only academic jobs check show in person. Now shows all 12 jobs, not just acadmic jobs that are also in person

Scenario 2: refresh page Select hiring center 1. See 2 jobs. check show in person. Should see 5 jobs, but see all 12. (note: If you check show in person, then select hiring center 1 you do properly see 5 jobs)

Scenario 3: after doing scenario 2: uncheck show in person. Should see 2 jobs. See 8.

Scenario 1: image

doodelicious commented 10 years ago

This issue was corrected with https://github.com/doodelicious/NotificationPortlet/commit/28b3734969b27fba460c156a03177fa9cc304793 however the jQuery function 'delegate' needed to be updated to 'on'.