Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Preparing for 5.2.0.beta2 release | Rafael Mendonça França | 2017-11-28 | 1 | -1/+1 |
| | |||||
* | Preparing for 5.2.0.beta1 release | Rafael Mendonça França | 2017-11-27 | 1 | -1/+1 |
| | |||||
* | Sort mailer previews | Dwight Watson | 2017-11-14 | 1 | -1/+1 |
| | |||||
* | Use .tt extension to all the template files | Rafael Mendonça França | 2017-11-13 | 2 | -0/+0 |
| | | | | | | | | Make clear that the files are not to be run for interpreters. Fixes #23847. Fixes #30690. Closes #23878. | ||||
* | [Action Mailer] require_relative => require | Akira Matsuda | 2017-10-21 | 2 | -4/+4 |
| | | | | This basically reverts cd9cc721ab54e2b0c7875cacf2113f03908a8bb7 | ||||
* | Add assert_enqueued_email_with to ActionMailer::TestHelper | Mikkel Malmberg | 2017-09-27 | 1 | -0/+42 |
| | |||||
* | Update Action Mailer doc [ci skip] | Yoshiyuki Hirano | 2017-08-30 | 3 | -8/+8 |
| | |||||
* | Use tt in doc for action_mailer [ci skip] | Yoshiyuki Hirano | 2017-08-26 | 1 | -1/+1 |
| | |||||
* | Merge pull request #29559 from kirs/eager-load-controller-actions | Rafael França | 2017-08-11 | 1 | -0/+6 |
|\ | | | | | Eager load controller actions to reduce response time of the first request | ||||
| * | Eager load controller and mailer actions | Kir Shatrov | 2017-07-29 | 1 | -0/+6 |
| | | | | | | | | | | | | | | | | | | | | | | On the first request, ActionController::Base#action_methods computes and memoized the list of available actions [1]. With this PR we move this expensive operation into eager load step to reduce response time of the first request served in production. This also reduces the memory footprint when running on forking server like Unicorn. [1] https://github.com/rails/rails/blob/a3813dce9a0c950a4af7909111fa730a2622b1db/actionpack/lib/abstract_controller/base.rb#L66-L77 | ||||
* | | Remove outdated comment [ci skip] | Youssef Boulkaid | 2017-08-06 | 1 | -4/+0 |
|/ | | | | | The comment was describing a previous version of the method with a different signature. This is outdated since e76c38e | ||||
* | Use frozen string literal in actionmailer/ | Kir Shatrov | 2017-07-23 | 18 | -0/+36 |
| | |||||
* | [Action Mailer] require => require_relative | Akira Matsuda | 2017-07-01 | 2 | -4/+4 |
| | |||||
* | Fix formatting of `ActionMailer::MessageDelivery` doc [ci skip] | Ryuta Kamizono | 2017-06-16 | 1 | -4/+4 |
| | |||||
* | Document setting the delivery_job for ActionMailer [ci skip] | Matthew Mongeau | 2017-06-16 | 1 | -0/+16 |
| | |||||
* | Allow mailers to configure their delivery job | Matthew Mongeau | 2017-06-15 | 2 | -1/+3 |
| | | | | | | | | | | | Setting delivery_job on a mailer class will cause MessageDelivery to use the specified job instead of ActionMailer::DeliveryJob: class MyMailer < ApplicationMailer self.delivery_job = MyCustomDeliveryJob ... end | ||||
* | Use mattr_accessor default: option throughout the project | Genadi Samokovarov | 2017-06-03 | 2 | -10/+4 |
| | |||||
* | Add option for class_attribute default (#29270) | David Heinemeier Hansson | 2017-05-29 | 2 | -6/+3 |
| | | | | | | | | | | | | * Allow a default value to be declared for class_attribute * Convert to using class_attribute default rather than explicit setter * Removed instance_accessor option by mistake * False is a valid default value * Documentation | ||||
* | Define path with __dir__ | bogdanvlviv | 2017-05-23 | 1 | -1/+1 |
| | | | | | | ".. with __dir__ we can restore order in the Universe." - by @fxn Related to 5b8738c2df003a96f0e490c43559747618d10f5f | ||||
* | Merge pull request #28835 from fphilipe/master | Rafael França | 2017-05-02 | 1 | -0/+1 |
|\ | | | | | Add missing require in ActionMailer::Base | ||||
| * | Add missing require in ActionMailer | Philipe Fatio | 2017-05-02 | 1 | -0/+1 |
| | | | | | | | | | | | | | | | | | | Without this, Action Mailer doesn't work like it used to on version 4. The following snippet fails since version 5: > require 'action_mailer' > ActionMailer::Base NameError: uninitialized constant ActiveSupport::Rescuable | ||||
* | | Merge pull request #28244 from ixti/improve/action-mailer-preview-params | Rafael França | 2017-04-26 | 1 | -2/+8 |
|\ \ | |/ |/| | Pass request params to ActionMailer::Preview | ||||
| * | Pass request params to ActionMailer::Preview | Alexey Zapparov | 2017-03-01 | 1 | -2/+8 |
| | | |||||
* | | Start Rails 5.2 development | Matthew Draper | 2017-03-22 | 1 | -2/+2 |
| | | |||||
* | | Document using `default_url_options` in an ActionMailer class. | Krzysztof Zych | 2017-03-08 | 1 | -0/+3 |
|/ | |||||
* | Preparing for 5.1.0.beta1 release | Rafael Mendonça França | 2017-02-23 | 1 | -1/+1 |
| | |||||
* | Freeze fragment cache related instrument name. | Stan Lo | 2017-02-07 | 1 | -1/+1 |
| | | | | | | | | | | | | | ActionMailer::Base#instrument_name and ActionController::Base#instrument_name will be frequently called once caching is enabled. So it's better to freeze them instead of create new string on every call. Also, the instrument name in #instrument_fragment_cache will usually be "write_fragment.action_controller" or "read_fragment.action_controller". So freezing them might also gain some performance improvement. We have done something like this in other places: https://github.com/rails/rails/blob/master/actionview/lib/action_view/template.rb#L348 | ||||
* | Add `:args` to `process.action_mailer` event | yuuji.yaginuma | 2017-02-04 | 1 | -1/+2 |
| | |||||
* | Don't mutate raw_source in mailer preview interceptor | Eugene Kenny | 2017-01-30 | 1 | -5/+1 |
| | | | | | | | | The raw_source method is documented as returning the exact value that was used to create the body; mutating it breaks that contract. Additionally, if the value used to create the body is blank, raw_source returns a frozen string which causes the interceptor to raise an error. | ||||
* | add default value to `deliver_later_queue_name` option [ci skip] | yuuji.yaginuma | 2017-01-31 | 1 | -1/+1 |
| | |||||
* | Document with with an example an link to Parameterized | Rafael Mendonça França | 2017-01-30 | 1 | -1/+6 |
| | | | | | Also change the class_methods to ClassMethods since the former document the method as an instance method of Parameterized not as a class method. | ||||
* | :scissors: | Rafael Mendonça França | 2017-01-30 | 1 | -1/+1 |
| | |||||
* | No need to advertise in the rdoc documentation | Rafael Mendonça França | 2017-01-30 | 1 | -3/+0 |
| | |||||
* | Implement respond_to_missing? in the Parameterized::Mailer class | Rafael Mendonça França | 2017-01-30 | 1 | -6/+11 |
| | |||||
* | Remove unneeded nodoc | Rafael Mendonça França | 2017-01-30 | 1 | -1/+1 |
| | |||||
* | Override the initializers instead of using tap | Rafael Mendonça França | 2017-01-30 | 1 | -3/+6 |
| | | | | We own the class so we can override the initialize. | ||||
* | Make internal classes internal for the documentation | Rafael Mendonça França | 2017-01-30 | 1 | -2/+2 |
| | |||||
* | Make assert_enqueued_emails + assert_no_enqueued_emails consider ↵ | David Heinemeier Hansson | 2017-01-30 | 1 | -2/+2 |
| | | | | | | parameterized delivery jobs Needed for testing of parameterized mailers | ||||
* | [ci skip] other -> order; expand ivar | Kasper Timm Hansen | 2017-01-28 | 1 | -1/+2 |
| | |||||
* | Offer the option to use parameterization for shared processing of headers ↵ | David Heinemeier Hansson | 2017-01-28 | 3 | -7/+148 |
| | | | | | and ivars (#27825) Offer the option to use parameterization for shared processing of headers and ivars | ||||
* | ZOMG worst typo in my life :scream: | Akira Matsuda | 2017-01-15 | 1 | -1/+1 |
| | |||||
* | `respond_to_missing?` should fallback to `super` where method_missing could ↵ | Akira Matsuda | 2017-01-15 | 1 | -1/+1 |
| | | | | call `super` | ||||
* | Reduce string objects by using \ instead of + or << for concatenating strings | Akira Matsuda | 2017-01-12 | 1 | -2/+2 |
| | | | | (I personally prefer writing one string in one line no matter how long it is, though) | ||||
* | Merge pull request #27227 from MQuy/allow-custom-content-type-in-mail-body | Rafael Mendonça França | 2017-01-06 | 1 | -4/+21 |
|\ | | | | | | | Allow to custom content type when setting mailer body | ||||
| * | Remove unnecessary condition in content_type | MQuy | 2017-01-06 | 1 | -1/+1 |
| | | |||||
| * | Add document in mailer | MQuy | 2016-12-06 | 1 | -0/+13 |
| | | |||||
| * | allow context type when set body mail | MQuy | 2016-11-30 | 1 | -4/+8 |
| | | |||||
* | | `self.` is not needed when calling its own instance method | Akira Matsuda | 2017-01-05 | 4 | -4/+4 |
| | | | | | | | | Actually, private methods cannot be called with `self.`, so it's not just redundant, it's a bad habit in Ruby | ||||
* | | Revert "Merge pull request #27550 from ↵ | Rafael Mendonça França | 2017-01-03 | 1 | -1/+1 |
| | | | | | | | | | | | | | | | | | | mtsmfm/fix-generator-command-for-nested-rails-engine" This reverts commit 1e969bfb98b88799e2c759fce25a1d8cf00d7ce7, reversing changes made to a5041f267ded119c2d00b8786c2f2c1e3f93c8a1. Reason: It breaks the public API | ||||
* | | Fix generator command for nested (namespaced) rails engine | Fumiaki MATSUSHIMA | 2017-01-03 | 1 | -1/+1 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | If we create nested (namespaced) rails engine such like bukkits-admin, `bin/rails g scaffold User name:string age:integer` will create `bukkits-admin/app/controllers/bukkits/users_controller.rb` but it should create `bukkits-admin/app/controllers/bukkits/admin/users_controller.rb`. In #6643, we changed `namespaced_path` as root path because we supposed application_controller is always in root but nested rails engine's application_controller will not. |