Skip to content


Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP


Fixes #1845 - Default assets registered twice by ActiveAdmin.setup #1974

wants to merge 1 commit into from

4 participants

Michael Shannon Matthew Millsaps-Brewer Olek Janiszewski Sean Linsley
Michael Shannon

Implements changes suggested in issue #1845 comments

Matthew Millsaps-Brewer

Will you please add a spec for this in spec/unit/asset_registration_spec.rb?

Olek Janiszewski

Cool! This is related: ejholmes/active_admin_editor#20

Sean Linsley

Our Travis builds on master are now green (as of 166bc46). Please rebase your PR on the current master:

# Expectations: "upstream" is gregbell's GitHub repo and "origin" is your fork

# Rebase your fork's master branch with the latest upstream changes
git checkout master
git pull --rebase upstream master
git push origin master

# Rebase your feature branch with the latest upstream changes
git checkout your_feature_branch
git pull --rebase upstream master
git push origin your_feature_branch # note that you may need to use -f
Sean Linsley

Ping @michaelrshannon; this still needs tests!

Michael Shannon

Thanks - been away, will deal with this week.

Sean Linsley

It's been two weeks now :P

Sean Linsley

Closing this in favor of #2139

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
This page is out of date. Refresh to see the latest.
Showing with 3 additions and 2 deletions.
  1. +3 −2 lib/active_admin/asset_registration.rb
5 lib/active_admin/asset_registration.rb
@@ -4,7 +4,8 @@ module AssetRegistration
# Stylesheets
def register_stylesheet(*args)
- stylesheets <<*args)
+ tmp =*args)
+ stylesheets << tmp unless stylesheets.include? tmp
def stylesheets
@@ -19,7 +20,7 @@ def clear_stylesheets!
# Javascripts
def register_javascript(name)
- javascripts << name
+ javascripts << name unless javascripts.include? name
def javascripts
Something went wrong with that request. Please try again.