aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* 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
| | | | |
| | * | | Merge pull request #3 from frodsan/patch-1George Claghorn2018-12-141-1/+1
| | |\ \ \ | | | |/ / | | |/| | Fix typo [ci skip]
| | | * | Fix typo [ci skip]Francesco Rodríguez2018-12-141-1/+1
| | |/ /
| | * | Merge pull request #2 from panckreous/patch-1George Claghorn2018-12-131-1/+1
| | |\ \ | | | | | | | | | | fix minor typo
| | | * | fix minor typoDan2018-12-131-1/+1
| | |/ / | | | | | | | | https://www.merriam-webster.com/words-at-play/top-10-commonly-confused-words-vol-2
| | * | Don't need to trip it up right awayDavid Heinemeier Hansson2018-12-131-1/+1
| | | |
| | * | Add installerDavid Heinemeier Hansson2018-12-133-9/+17
| | | |
| | * | Tidy up and example testingDavid Heinemeier Hansson2018-12-131-3/+27
| | | |
| | * | SimplerDavid Heinemeier Hansson2018-12-131-3/+1
| | | |
| | * | Document the ingress controllersGeorge Claghorn2018-12-136-1/+168
| | | |
| | * | Document ingress configurationGeorge Claghorn2018-12-131-6/+147
| | | |
generated by cgit v1.2.3 (git 2.25.1) at 2024-12-26 20:54:59 +0000