-
-
Notifications
You must be signed in to change notification settings - Fork 487
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
Rails upgrade 7.0.8 to 7.1.0 #5478
Changes from 12 commits
123c200
6c2f97e
23befdf
c7b9f24
78bd955
67204da
33edac1
49f45e8
097686a
7749518
c6d07d8
d267aa9
63e833a
1d639db
7577090
d7b4b1f
5506197
6e57ee4
6d9e514
1a05d0d
2f7ee4d
52b5a59
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -11,6 +11,11 @@ class Application < Rails::Application | |
# Initialize configuration defaults for originally generated Rails version. | ||
config.load_defaults 7.0 | ||
|
||
# Please, add to the `ignore` list any other `lib` subdirectories that do | ||
# not contain `.rb` files, or that should not be reloaded or eager loaded. | ||
# Common ones are `templates`, `generators`, or `middleware`, for example. | ||
config.autoload_lib(ignore: %w[assets tasks]) | ||
|
||
# Configuration for the application, engines, and railties goes here. | ||
# | ||
# These settings can be overridden in specific environments using the files | ||
|
@@ -19,7 +24,8 @@ class Application < Rails::Application | |
# config.time_zone = "Central Time (US & Canada)" | ||
# config.eager_load_paths << Rails.root.join("extras") | ||
|
||
config.action_mailer.preview_path ||= defined?(Rails.root) ? Rails.root.join("lib", "mailers", "previews") : nil | ||
config.action_mailer.preview_paths ||= [] | ||
config.action_mailer.preview_paths << (defined?(Rails.root) ? Rails.root.join("lib", "mailers", "previews") : nil) | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I'm wondering if the difference between There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. In Rails 7.1.0 there is now support for multiple preview paths so I am appending as per new configuration. In one of the earlier commit, I still reverted to earlier code base to check if specs pass with it but encountered same issue so not sure if this is the problem 1d639db |
||
config.eager_load_paths << Rails.root.join("app", "lib", "importers") | ||
config.assets.paths << Rails.root.join("app", "assets", "webfonts") | ||
config.active_storage.variant_processor = :mini_magick | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Had to remove rails_blob_path and update with url code to handle ActionView::Template::Error: Cannot get a signed_id for a new record error.
Referred this open issue - https://stackoverflow.com/questions/77201333/cannot-get-a-signed-id-for-a-new-record-with-url-for-after-attaching-pictur