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

Root route required by sidebar #32

Closed
wants to merge 1 commit into from

Conversation

pablobm
Copy link
Contributor

@pablobm pablobm commented Mar 14, 2020

Since thoughtbot/administrate#1376, Administrate requires a root route by default. Otherwise it raises the following:

ActionView::Template::Error (undefined local variable or method `root_url' for #<#<Class:0x000055d0d8b83f40>:0x000055d0d8b81380>):
     8: %>
     9: 
    10: <nav class="navigation" role="navigation">
    11:   <%= link_to "Back to app", root_url, class: "button button--alt" %> 
    12: 
    13:   <% Administrate::Namespace.new(namespace).resources.each do |resource| %>
    14:     <%= link_to(
  
administrate (0.13.0) app/views/administrate/application/_navigation.html.erb:11
...

This PR adds this route, so that the dummy app can run.

@sedubois
Copy link
Contributor

Duplicate of #31 🙂

@pablobm
Copy link
Contributor Author

pablobm commented Mar 23, 2020

Ooops! 😅

@pablobm pablobm closed this Mar 23, 2020
@pablobm pablobm deleted the root-required branch March 23, 2020 18:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants