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

Update AdminLTE to 2.4.15 #1015

Merged
merged 10 commits into from
Jul 17, 2019

Conversation

amosfolz
Copy link
Contributor

@amosfolz amosfolz commented Jul 7, 2019

Will need help identifying if anything in AdminLTE-custom.js that I removed is actually still needed. (As far as I can tell, it is not)

issue #994

@amosfolz amosfolz changed the title Develop update admin lte 2.4.12 Update AdminLTE to 2.4.12 Jul 7, 2019
@amosfolz amosfolz marked this pull request as ready for review July 10, 2019 20:23
@amosfolz
Copy link
Contributor Author

I just looked this over and manually tested again. Pending if @lcharette can remember what these lines were meant to fix, this is ready for merge.

@lcharette
Copy link
Member

Pending if @lcharette can remember what these lines were meant to fix, this is ready for merge.

https://github.com/userfrosting/UserFrosting/blob/master/app/sprinkles/core/assets/userfrosting/js/AdminLTE-custom.js#L59

Overwrite here. Otherwise, the menu href based active will be removed

Not sure what that meant, but I believe it's linked to nested entries in the nav menu. Might be something that's been solved in newer version of AdminLTE.

@amosfolz
Copy link
Contributor Author

Okay, I'll test out a nested menu to make sure it's working right.

@amosfolz
Copy link
Contributor Author

Just putting this here for reference https://adminlte.io/docs/2.4/upgrade-guide

@codecov
Copy link

codecov bot commented Jul 11, 2019

Codecov Report

Merging #1015 into develop will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             develop    #1015   +/-   ##
==========================================
  Coverage      66.93%   66.93%           
  Complexity      1913     1913           
==========================================
  Files            159      159           
  Lines           6684     6684           
==========================================
  Hits            4474     4474           
  Misses          2210     2210

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update df8495e...f999433. Read the comment docs.

@codecov
Copy link

codecov bot commented Jul 11, 2019

Codecov Report

Merging #1015 into develop will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             develop    #1015   +/-   ##
==========================================
  Coverage      66.96%   66.96%           
  Complexity      1918     1918           
==========================================
  Files            160      160           
  Lines           6693     6693           
==========================================
  Hits            4482     4482           
  Misses          2211     2211

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 66df775...6105a86. Read the comment docs.

@amosfolz amosfolz changed the title Update AdminLTE to 2.4.12 Update AdminLTE to 2.4.13 Jul 15, 2019
@amosfolz
Copy link
Contributor Author

I tested the tree-view and everything worked OK.

@amosfolz amosfolz changed the title Update AdminLTE to 2.4.13 Update AdminLTE to 2.4.14 Jul 16, 2019
@lcharette
Copy link
Member

They are making fun of you !

https://github.com/ColorlibHQ/AdminLTE/releases/tag/v2.4.15

Only version updated.
@amosfolz amosfolz changed the title Update AdminLTE to 2.4.14 Update AdminLTE to 2.4.15 Jul 16, 2019
@lcharette
Copy link
Member

Alright, let's freeze it on 2.4.15 for now

@lcharette lcharette merged commit a2f8304 into userfrosting:develop Jul 17, 2019
@amosfolz amosfolz deleted the develop-updateAdminLTE-2.4.12 branch July 17, 2019 01:10
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

2 participants