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

fix uninitialized constant InheritedResources::Base (NameError) #23

Closed
wants to merge 1 commit into from

Conversation

mvaragnat
Copy link

No description provided.

@mvaragnat
Copy link
Author

based on this advice : activeadmin/activeadmin#7196 (comment)

@ghulamjilani
Copy link

ghulamjilani commented Jan 18, 2023

I upgraded my rails version from 6.1 to 7 but facing the following issue related to activeadmin.
I have Implemented many solutions including Rails.application.config.after_initialize do
But, still it is throwing this error..

uninitialized constant InheritedResources::Base (NameError)

  class BaseController < ::InheritedResources::Base
                                             ^^^^^^
Did you mean?  Base64

@Ivan-P23994
Copy link

I upgraded my rails version from 6.1 to 7 but facing the following issue related to activeadmin. I have Implemented many solutions including Rails.application.config.after_initialize do But, still it is throwing this error..

uninitialized constant InheritedResources::Base (NameError)

  class BaseController < ::InheritedResources::Base
                                             ^^^^^^
Did you mean?  Base64

Have you managed to find a workaround? I am encountering the same issue...

@FlintOFF
Copy link

Same issue

@rbngzlv
Copy link

rbngzlv commented May 22, 2023

I opened a new PR with the changes that worked for me (#25), because with this changes another errors is raised. I added a comment in the PR description with a workaround while we wait for someane to merge the PR.

EDIT: The pull request #22 also proposes the use of the ActiveAdmin.before_load, so I've closed mine in favour of the older one. If #22 is merged, this PR is also not needed.

@mvaragnat
Copy link
Author

closing in favor of #22

#22 solves the problems when upgrading to Rails 7

@mvaragnat mvaragnat closed this Aug 18, 2023
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

5 participants