Skip to content

Rails 7 - Wrap BankApiService in reloader callback #3250

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

Merged
merged 0 commits into from
Nov 25, 2022

Conversation

albertchae
Copy link
Contributor

#3111

This fixes this deprecation warning which is an error in Rails 7

DEPRECATION WARNING: Initialization autoloaded the constants Util and BankApiService.

Being able to do this is deprecated. Autoloading during initialization is going
to be an error condition in future versions of Rails.

Reloading does not reboot the application, and therefore code executed during
initialization does not run again. So, if you reload Util, for example,
the expected changes won't be reflected in that stale Module object.

These autoloaded constants have been unloaded.

In order to autoload safely at boot time, please wrap your code in a reloader
callback this way:

    Rails.application.reloader.to_prepare do
      # Autoload classes and modules needed at boot time here.
    end

That block runs when the application boots, and every time there is a reload.
For historical reasons, it may run twice, so it has to be idempotent.

Check the "Autoloading and Reloading Constants" guide to learn more about how
Rails autoloads and reloads.
 (called from <top (required)> at /usr/src/app/config/environment.rb:7)

@albertchae albertchae changed the title Wrap BankApiService in reloader callback Rails 7 - Wrap BankApiService in reloader callback Nov 23, 2022
@albertchae albertchae force-pushed the rails-7.autoloading-fix branch from ba4868c to 4cd5c20 Compare November 25, 2022 10:45
@albertchae albertchae enabled auto-merge (squash) November 25, 2022 10:45
@albertchae albertchae temporarily deployed to bank-hackclu-rails-7-au-whrdmt November 25, 2022 10:46 Inactive
@albertchae albertchae merged commit b6bf6a9 into main Nov 25, 2022
@albertchae albertchae deleted the rails-7.autoloading-fix branch November 25, 2022 10:47
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.

5 participants