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

Apply fixes from StyleCI #4645

Merged
merged 1 commit into from Dec 31, 2018
Merged

Apply fixes from StyleCI #4645

merged 1 commit into from Dec 31, 2018

Conversation

DawoudIO
Copy link
Contributor

This pull request applies code style fixes from an analysis carried out by StyleCI.


For more information, click here.

@ghost ghost assigned DawoudIO Dec 31, 2018
@ghost ghost added the In Review label Dec 31, 2018
@DawoudIO DawoudIO merged commit 1ae3a6c into dev/adminlte-2.4.x Dec 31, 2018
@DawoudIO DawoudIO deleted the analysis-zdyBe7 branch December 31, 2018 21:15
@ghost ghost removed the In Review label Dec 31, 2018
DawoudIO added a commit that referenced this pull request Jan 2, 2019
* npm install working now

* new adminlte.js file name

* updated per adminlTE upgrade

* working on addming removed plugsins via Grunt

* removed skin/adminlte

* working login page without css/js errors

* Apply fixes from StyleCI (#4640)

* working login page without css/js errors

* working pages without css/js errors

* npm audit updates

* removed sSwfPath as it is no longer supported

* moved dom outside isCSV check

* Apply fixes from StyleCI (#4645)

* more datatable buttons js/css

* fixed issues with CVS export perm not working for admin users

* working datatable export with new combine js /css

* better security and datatable downoads

* Apply fixes from StyleCI (#4646)

* Update FamilyVerify.js

* cleanner Export buttons

* fixed datatable js files

* using jQuery extend to merge datatable config

* removed npm download of datatable as we get it from the build system

* fixed missing images for Datatables and ensure all are using the same version

* updated lock file after merge
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant