upgraded and consolidated js datatables(and extensions) packages #427

Merged
merged 1 commit into from Jan 11, 2017

Projects

None yet

2 participants

@bradymiller
Member
bradymiller commented Jan 10, 2017 edited

TODO:
Test the Zend patient validator.
Look through the bower packages to ensure nothing needs to be remove (.gitignore stuff etc.)

@bradymiller bradymiller referenced this pull request Jan 10, 2017
Open

Dynamic Finder #425

@bradymiller bradymiller changed the title from upgraded and consolidated js datatables package to upgraded and consolidated js datatables(and extensions) packages Jan 10, 2017
-<style type="text/css">
-@import "../../../library/js/datatables/media/css/demo_page.css";
-@import "../../../library/js/datatables/media/css/demo_table.css";
-.mytopdiv { float: left; margin-right: 1em; }
@bradymiller
bradymiller Jan 10, 2017 Member

may need to add the above mytopdiv style line back

@bradymiller
bradymiller Jan 11, 2017 Member

didn't do anything on testing, so left it out

@bradymiller
Member

Testing well, so will bring this in. Will also bring into rel-500, since this is a bug fix for the Zend patient validator module.

@bradymiller bradymiller merged commit a374d43 into openemr:master Jan 11, 2017
@sunsetsystems
Contributor

Cool. I'm a bit surprised this works since Datatables 1.10 changed the whole API.

@bradymiller
Member

The only hang up was actually the newer jquery verion which no longer supported the .live function.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment