Zeitwerk::NameError in Madmin::DashboardController#show

Question

Zeitwerk::NameError in Madmin::DashboardController#show

timmyd87 opened this issue 3 years ago · comments

Hi team,

I ran the install command on a semi changed jump start pro app. It builds out all the resources and seems to get the routes right however i get the error 'Zeitwerk::NameError in Madmin::DashboardController#show' saying it expected 'to define constant Resources::AccountInvitationResource, but didn't' - account invitation is the example here. Adding 'Resources::' to the class name just throws another error.

Any clues? A little wary of running install again unless there are other options to pass in?

Chris Oliver · Answer 1 · Wed Apr 21 2021 00:47:31 GMT+0800 (China Standard Time)

I haven't tested this on Jumpstart Pro yet, but I will soon and will get this fixed. 👍

spacerobotTR · Answer 2 · Wed Apr 21 2021 23:22:48 GMT+0800 (China Standard Time)

I have a similar error on jumpstart github. When I first start foreman and go to the page I get this error:

expected file /Users/xxx/Documents/Sites/madmintest/app/madmin/resources/action_text/rich_text_resource.rb to define constant ActionText::RichTextResource, but didn't

If i hit refresh it loads fine and doesn't come up again. Any ideas on what this is?

Chris Oliver · Answer 3 · Fri Apr 23 2021 00:19:54 GMT+0800 (China Standard Time)

@spacerobotTR can you paste me the contents of the rich_text_resource.rb file?

I just verified it generated it fine for the test dummy app in the madmin repo, so not sure why yours would be different.

spacerobotTR · Answer 4 · Fri Apr 23 2021 00:38:56 GMT+0800 (China Standard Time)

It is actually empty with nothing in it.

…

On Thu, Apr 22, 2021 at 12:20 PM Chris Oliver ***@***.***> wrote: @spacerobotTR <https://github.com/spacerobotTR> can you paste me the contents of the rich_text_resource.rb file? I just verified it generated it fine for the test dummy app in the madmin repo, so not sure why yours would be different. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#46 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AA4725F7XAHW7LIVEIUE4TLTKBED7ANCNFSM43D7Y4IQ> .

Chris Oliver · Answer 5 · Fri Apr 23 2021 02:03:47 GMT+0800 (China Standard Time)

@timmyd87 I don't know why, but I started getting that error too. If I refresh the page, it resolves itself immediately without any changes. Strange.

And yet, I can enable the logger and it loads just fine after a reboot.

Zeitwerk@rails.main: constant PostResource loaded from file /Users/chris/code/madmin/test/dummy/app/madmin/resources/post_resource.rb

Possibly related to the eager_load paths that we're adding to the Rails app to make app/madmin/resources work.

timmyd87 · Answer 6 · Fri Apr 23 2021 15:28:28 GMT+0800 (China Standard Time)

Thanks @excid3 for taking a look. I can confirm if i refresh the screen it will load without any changes. If you have multiple resources you have to refresh until it's moved through each resource before it shows without error.

Chris Oliver · Answer 7 · Fri Apr 23 2021 21:42:11 GMT+0800 (China Standard Time)

If you have multiple resources you have to refresh until it's moved through each resource before it shows without error.

Interesting, I did not have that issue. For me, it was only the PostResource but everything else loaded fine.

And then coming back after a few hours, it's not doing it again. I wonder if there is something being cached with bootsnap or something? Maybe try clearing the tmp directory?

If anyone can find a reliable way to reproduce this, let me know.

spacerobotTR · Answer 8 · Fri Apr 23 2021 21:50:00 GMT+0800 (China Standard Time)

You have to bare with me as I am not a rails pro or anything. I did a clean install on another new project. I don't seem to be getting that error now. I also noticed I do not have the resources/action text/rich_text_resource on this install. Not sure why, but saw this in the console. Not sure what it means. Skipping ActionMailbox::InboundEmail because database table does not exist Skipping ActionText::RichText because database table does not exist

…

On Fri, Apr 23, 2021 at 9:42 AM Chris Oliver ***@***.***> wrote: If you have multiple resources you have to refresh until it's moved through each resource before it shows without error. Interesting, I did not have that issue. For me, it was only the PostResource but everything else loaded fine. And then coming back after a few hours, it's not doing it again. I wonder if there is something being cached with bootsnap or something? Maybe try clearing the tmp directory? If anyone can find a reliable way to reproduce this, let me know. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#46 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AA4725AYNHSQOSMBFVOYCWDTKF2MNANCNFSM43D7Y4IQ> .

Chris Oliver · Answer 9 · Sat Apr 24 2021 00:20:08 GMT+0800 (China Standard Time)

It skips them because you don't have ActionMailbox or ActionText installed.

spacerobotTR · Answer 10 · Sat Apr 24 2021 00:23:13 GMT+0800 (China Standard Time)

That is not part of the jumpstart install template?

…

On Fri, Apr 23, 2021 at 12:20 PM Chris Oliver ***@***.***> wrote: It skips them because you don't have ActionMailbox or ActionText installed. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#46 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AA4725HKZWOMS3AKYY5UHMDTKGM4VANCNFSM43D7Y4IQ> .

Chris Oliver · Answer 11 · Tue May 18 2021 04:35:39 GMT+0800 (China Standard Time)

I just rewrote this to read through the files instead of using eager_load. Should be more reliable and doesn't require loading your entire application. 👍