aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Add missing quotes on Mailbox test templateyuuji.yaginuma2018-12-281-1/+1
|
* Don't load Action Mailbox when Active Record is skippedGeorge Claghorn2018-12-282-2/+7
|
* Load Action Mailbox when other components are skippedGeorge Claghorn2018-12-281-0/+1
|
* Deprecate ActionMailer::Base.receive in favor of Action MailboxGeorge Claghorn2018-12-275-40/+14
|
* Use high level API on `migration_context` instead of using low level API ↵Ryuta Kamizono2018-12-282-13/+11
| | | | | | | | directly Since `migration_context` has `migrations_paths` itself and provides methods which returning values from parsed migration files, so there is no reason to use the `parse_migration_filename` low level API directly.
* Deprecate passing `migrations_paths` to ↵Ryuta Kamizono2018-12-284-12/+20
| | | | | | | `connection.assume_migrated_upto_version` Since #31727, `migrations_paths` in `assume_migrated_upto_version` is no longer used.
* Remove redundant assignning to `current_env`Ryuta Kamizono2018-12-281-1/+0
| | | | This was introduced at https://github.com/rails/rails/commit/cfa1df4b07bee5b2bbcbf9edd2ac287b4fb23c18#diff-b36b9c41be30b05dc14d09d7f3b192efR436.
* Enable `Lint/ShadowingOuterLocalVariable` cop to avoid newly adding the warningRyuta Kamizono2018-12-281-0/+3
| | | | | | | | | | | Since we've addressed the warning "warning: shadowing outer local variable" frequently. 2c325182b84ea4c75855c777b25de0f15fd218e5 df76eaa4f1619dfbc024235f444c602eb6d6775a b86c2a6767b939c420687db7df078625c702dc7a b658743ac2a69d196d283e780816f5ad4a305753 b18f2fe96debdcd0744f99101d1a1e0d42c41eaa
* Merge pull request #34810 from cbothner/activestorage-no-undefined-csrf-headerGeorge Claghorn2018-12-273-2/+15
|\ | | | | ActiveStorage: Don’t include an undefined X-CSRF-Token header when creating a blob record
| * Don’t include an undefined X-CSRF-Token headerCameron Bothner2018-12-273-2/+15
|/ | | | | | | If there is not a `csrf-token` meta tag in the document, the blob record XHR was including an `X-CSRF-Token` header set to the string "undefined." Instead of setting it to undefined, it should not be included in the absence of a meta tag.
* Merge pull request #34806 from bogdan/reuse-find-targetRyuta Kamizono2018-12-274-30/+20
|\ | | | | | | Reuse AR::Association#find_target method
| * Reuse AR::Association#find_target methodBogdan Gusiev2018-12-274-29/+21
| |
* | Only run isolated tests on the latest stable ruby: that's now 2.6Ryuta Kamizono2018-12-271-1/+1
| | | | | | | | | | | | Follow up #34795. See also f1a30d8ad5ae17e6202fd3a126027378773dcaea.
* | Merge pull request #34801 from bogdanvlviv/follow-up-34786George Claghorn2018-12-266-52/+5
|\ \ | | | | | | Follow up 34786
| * | Fix Ruby warnings in actionmailboxbogdanvlviv2018-12-271-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ``` rails/actionmailbox$ bundle exec rake tests (snip) ... rails/activerecord/lib/active_record/persistence.rb:48: warning: in `create!': the last argument was passed as a single Hash rails/actionmailbox/app/models/action_mailbox/inbound_email/message_id.rb:21: warning: although a splat keyword arguments here ... (snip) ```
| * | Improve actionmailbox's .gitignore and remove redundant filesbogdanvlviv2018-12-275-51/+4
|/ / | | | | | | Follow up #34786
* | Nest ActionMailbox::Base in the API docs (missed in 6c168aa)George Claghorn2018-12-261-93/+95
| |
* | Nest Action Mailbox classes in the API docsGeorge Claghorn2018-12-2619-508/+546
| |
* | Generate Action Mailbox's API docsGeorge Claghorn2018-12-261-0/+8
| |
* | Start an Action Mailbox changelogGeorge Claghorn2018-12-261-0/+3
| |
* | Test against Ruby 2.6.0George Claghorn2018-12-261-0/+36
| |
* | Merge pull request #34786 from georgeclaghorn/actionmailboxGeorge Claghorn2018-12-26162-182/+10911
|\ \ | | | | | | Import Action Mailbox
| * \ Import Action MailboxGeorge Claghorn2018-12-25162-182/+10911
| |\ \
| | * | Bump the minimum Ruby version to match Rails masterGeorge Claghorn2018-12-201-1/+1
| | | |
| | * | Remove Ruby 2.4 from the test matrixGeorge Claghorn2018-12-191-1/+0
| | | | | | | | | | | | | | | | Rails 6 will require Ruby 2.5 or newer. See rails/rails#34754.
| | * | Use class_methods throughout codebase.Kasper Timm Hansen2018-12-192-4/+4
| | | |
| | * | [ci skip] Docs: fix spelling, routing setup call and formatting.Kasper Timm Hansen2018-12-192-9/+9
| | | |
| | * | Merge pull request #12 from dixpac/add_logger_when_auto_generating_message_idKasper Timm Hansen2018-12-191-4/+4
| | |\ \ | | | | | | | | | | Added logging when Message ID wasn't extracted
| | | * | Added logging when Message ID wasn't extractedDino Maric2018-12-191-4/+4
| | |/ /
| | * | Resurrect installer.Kasper Timm Hansen2018-12-184-3/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Running `./bin/rails generate mailbox application --no-test-framework` generates: ``` class ApplicationMailbox < ApplicationMailbox def process end end ``` which is not correct for the application mailbox. It shouldn't respond to process but it should contain a routing hint. Generally generators aren't meant to be used like the previous commit. The mailbox generator can certainly add in the ApplicationMailbox if missing, but it shouldn't be called with "application" as an argument. Also adds back auto inserting an `ingress` config line in `config/environmnets/production.rb`. Fixes #13. [Kasper Timm Hansen, Andrew Babichev]
| | * | Refit generator file structure to match Action Cable.Kasper Timm Hansen2018-12-187-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fits with rails/generators/channel containing: - channel_generator.rb - templates/ - USAGE
| | * | Remove actions argument from Mailbox generator.Kasper Timm Hansen2018-12-183-15/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Generators only have the public `process` action, so there's nothing else to generate here. Skip the needless indirection.
| | * | Test namespacing is supported.Kasper Timm Hansen2018-12-181-0/+22
| | | |
| | * | Include proper suffix to assert no file correctly.Kasper Timm Hansen2018-12-181-1/+2
| | | |
| | * | Merge pull request #10 from dixpac/subtile_generator_mention_in_readmeDavid Heinemeier Hansson2018-12-161-0/+5
| | |\ \ | | | | | | | | | | Subtile mention of generator inside README
| | | * | Subtile mention of generator inside READMEDino Maric2018-12-161-0/+5
| | | | | | | | | | | | | | | | | | | | Just a small mention that we can use generators inside README.
| | * | | Merge pull request #11 from dixpac/fix_required_ruby_versionGeorge Claghorn2018-12-161-1/+1
| | |\ \ \ | | | |/ / | | |/| | Set ruby >= 2.4.1 as min required version
| | | * | Set ruby >= 2.4.1 as min required versionDino Maric2018-12-161-1/+1
| | |/ / | | | | | | | | | | | | This matches Rails master, and it should make TravisCI green.
| | * | Hook up Travis CIGeorge Claghorn2018-12-161-0/+18
| | | |
| | * | Merge pull request #9 from dixpac/dix/fix_generatorsDavid Heinemeier Hansson2018-12-158-17/+26
| | |\ \ | | | | | | | | | | Fix Rails generators
| | | * | Fix Rails generatorsDino Maric2018-12-158-17/+26
| | |/ / | | | | | | | | | | | | | | | | 1.Don't generate ApplicationMailboxTest when executing installer 2. Hookup test_unit, so console doesn't throw errors
| | * | Merge pull request #8 from dgilperez/adds-generatorDavid Heinemeier Hansson2018-12-149-8/+154
| | |\ \ | | | | | | | | | | Add Mailbox and MailboxTest generators
| | | * | renames file namesDavid Gil2018-12-142-1/+1
| | | | |
| | | * | Uses the new generator to install ApplicationMailbox in ↵David Gil2018-12-143-13/+3
| | | | | | | | | | | | | | | | | | | | action_mailbox:install rake
| | | * | Add Mailbox and MailboxTest generatorsDavid Gil2018-12-147-0/+156
| | |/ /
| | * | Use the correct route methodDavid Heinemeier Hansson2018-12-141-1/+1
| | | |
| | * | Merge pull request #5 from kylekeesling/rails522-conductor-fixGeorge Claghorn2018-12-141-1/+1
| | |\ \ | | | | | | | | | | Development conductor ingress does not work with Rails 5.2.2
| | | * | remove unimplemented arguement when adding mail message to ActiveStorageKyle Keesling2018-12-141-1/+1
| | |/ / | | | | | | | | | | | | The indentify parameter is not yet implemented in Rails 5.2.2
| | * | Merge pull request #4 from dixpac/dix/add_frozen_string_literalsGeorge Claghorn2018-12-1451-0/+102
| | |\ \ | | | | | | | | | | Add frozen_string_literal: true to match Rails codebase
| | | * | Add frozen_string_literal: true to match Rails codebaseDino Maric2018-12-1451-0/+102
| | | | |