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.

Sign up for GitHub

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

Jump to bottom

Fix reorder URL when ActiveAdmin namespace is root #6

Open

ivanyv wants to merge 1 commit into dkniffin:master from ivanyv:fix_root_aa_namespace

ivanyv commented Dec 7, 2017

When you have an ActiveAdmin config with:

config.namespace false do |admin|
  # ...
end

The reorder URL generation will raise an error, given there's no actual namespace being used.


          Fix reorder URL when ActiveAdmin namespace is root

10a0dd2

ivanyv force-pushed the fix_root_aa_namespace branch from d0fb219 to 10a0dd2 Compare

December 7, 2017 21:39

Owner

dkniffin commented Jul 11, 2024

@ivanyv is this still valid? If so, can you please rebase to get the new test suite and fix the conflict?

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

Labels

None yet