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

ENHANCEMENT use SilverStripe form to serve up the filter form #72

Closed
1 of 2 tasks
muskie9 opened this issue Dec 18, 2015 · 1 comment · Fixed by #88
Closed
1 of 2 tasks

ENHANCEMENT use SilverStripe form to serve up the filter form #72

muskie9 opened this issue Dec 18, 2015 · 1 comment · Fixed by #88

Comments

@muskie9
Copy link
Member

muskie9 commented Dec 18, 2015

  • Use SilverStripe form for filter
  • Do filter processing via the controller rather than the js file
@jsirish jsirish added this to the 2.0.0 milestone Dec 26, 2015
@jsirish jsirish modified the milestones: 2.0.0-beta1, 2.0.0 Apr 7, 2016
jsirish added a commit to jsirish/silverstripe-locator that referenced this issue Apr 8, 2016
use SilverStripe to filter locations, rather than JS script

fixes dynamic#72
jsirish added a commit to jsirish/silverstripe-locator that referenced this issue Apr 9, 2016
use SilverStripe to filter locations, rather than JS script

fixes dynamic#72
jsirish added a commit to jsirish/silverstripe-locator that referenced this issue Apr 9, 2016
use SilverStripe to filter locations, rather than JS script

fixes dynamic#72
jsirish added a commit to jsirish/silverstripe-locator that referenced this issue Apr 9, 2016
use SilverStripe to filter locations, rather than JS script

fixes dynamic#72
muskie9 pushed a commit to jsirish/silverstripe-locator that referenced this issue Jul 29, 2016
use SilverStripe to filter locations, rather than JS script

fixes dynamic#72
@muskie9
Copy link
Member Author

muskie9 commented Aug 3, 2016

closed via #108

@muskie9 muskie9 closed this as completed Aug 3, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants