Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
| | * | | | | | | | Use class_methods throughout codebase. | Kasper Timm Hansen | 2018-12-19 | 2 | -4/+4 | |
| | | | | | | | | | ||||||
| | * | | | | | | | [ci skip] Docs: fix spelling, routing setup call and formatting. | Kasper Timm Hansen | 2018-12-19 | 2 | -9/+9 | |
| | | | | | | | | | ||||||
| | * | | | | | | | Merge pull request #12 from dixpac/add_logger_when_auto_generating_message_id | Kasper Timm Hansen | 2018-12-19 | 1 | -4/+4 | |
| | |\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | Added logging when Message ID wasn't extracted | |||||
| | | * | | | | | | | Added logging when Message ID wasn't extracted | Dino Maric | 2018-12-19 | 1 | -4/+4 | |
| | |/ / / / / / / | ||||||
| | * | | | | | | | Resurrect installer. | Kasper Timm Hansen | 2018-12-18 | 4 | -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 Hansen | 2018-12-18 | 7 | -2/+2 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fits with rails/generators/channel containing: - channel_generator.rb - templates/ - USAGE | |||||
| | * | | | | | | | Remove actions argument from Mailbox generator. | Kasper Timm Hansen | 2018-12-18 | 3 | -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 Hansen | 2018-12-18 | 1 | -0/+22 | |
| | | | | | | | | | ||||||
| | * | | | | | | | Include proper suffix to assert no file correctly. | Kasper Timm Hansen | 2018-12-18 | 1 | -1/+2 | |
| | | | | | | | | | ||||||
| | * | | | | | | | Merge pull request #10 from dixpac/subtile_generator_mention_in_readme | David Heinemeier Hansson | 2018-12-16 | 1 | -0/+5 | |
| | |\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | Subtile mention of generator inside README | |||||
| | | * | | | | | | | Subtile mention of generator inside README | Dino Maric | 2018-12-16 | 1 | -0/+5 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Just a small mention that we can use generators inside README. | |||||
| | * | | | | | | | | Merge pull request #11 from dixpac/fix_required_ruby_version | George Claghorn | 2018-12-16 | 1 | -1/+1 | |
| | |\ \ \ \ \ \ \ \ | | | |/ / / / / / / | | |/| | | | | | | | Set ruby >= 2.4.1 as min required version | |||||
| | | * | | | | | | | Set ruby >= 2.4.1 as min required version | Dino Maric | 2018-12-16 | 1 | -1/+1 | |
| | |/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | This matches Rails master, and it should make TravisCI green. | |||||
| | * | | | | | | | Hook up Travis CI | George Claghorn | 2018-12-16 | 1 | -0/+18 | |
| | | | | | | | | | ||||||
| | * | | | | | | | Merge pull request #9 from dixpac/dix/fix_generators | David Heinemeier Hansson | 2018-12-15 | 8 | -17/+26 | |
| | |\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | Fix Rails generators | |||||
| | | * | | | | | | | Fix Rails generators | Dino Maric | 2018-12-15 | 8 | -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-generator | David Heinemeier Hansson | 2018-12-14 | 9 | -8/+154 | |
| | |\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | Add Mailbox and MailboxTest generators | |||||
| | | * | | | | | | | renames file names | David Gil | 2018-12-14 | 2 | -1/+1 | |
| | | | | | | | | | | ||||||
| | | * | | | | | | | Uses the new generator to install ApplicationMailbox in ↵ | David Gil | 2018-12-14 | 3 | -13/+3 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | action_mailbox:install rake | |||||
| | | * | | | | | | | Add Mailbox and MailboxTest generators | David Gil | 2018-12-14 | 7 | -0/+156 | |
| | |/ / / / / / / | ||||||
| | * | | | | | | | Use the correct route method | David Heinemeier Hansson | 2018-12-14 | 1 | -1/+1 | |
| | | | | | | | | | ||||||
| | * | | | | | | | Merge pull request #5 from kylekeesling/rails522-conductor-fix | George Claghorn | 2018-12-14 | 1 | -1/+1 | |
| | |\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | Development conductor ingress does not work with Rails 5.2.2 | |||||
| | | * | | | | | | | remove unimplemented arguement when adding mail message to ActiveStorage | Kyle Keesling | 2018-12-14 | 1 | -1/+1 | |
| | |/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | The indentify parameter is not yet implemented in Rails 5.2.2 | |||||
| | * | | | | | | | Merge pull request #4 from dixpac/dix/add_frozen_string_literals | George Claghorn | 2018-12-14 | 51 | -0/+102 | |
| | |\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | Add frozen_string_literal: true to match Rails codebase | |||||
| | | * | | | | | | | Add frozen_string_literal: true to match Rails codebase | Dino Maric | 2018-12-14 | 51 | -0/+102 | |
| | | | | | | | | | | ||||||
| | * | | | | | | | | Merge pull request #3 from frodsan/patch-1 | George Claghorn | 2018-12-14 | 1 | -1/+1 | |
| | |\ \ \ \ \ \ \ \ | | | |/ / / / / / / | | |/| | | | | | | | Fix typo [ci skip] | |||||
| | | * | | | | | | | Fix typo [ci skip] | Francesco Rodríguez | 2018-12-14 | 1 | -1/+1 | |
| | |/ / / / / / / | ||||||
| | * | | | | | | | Merge pull request #2 from panckreous/patch-1 | George Claghorn | 2018-12-13 | 1 | -1/+1 | |
| | |\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | fix minor typo | |||||
| | | * | | | | | | | fix minor typo | Dan | 2018-12-13 | 1 | -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 away | David Heinemeier Hansson | 2018-12-13 | 1 | -1/+1 | |
| | | | | | | | | | ||||||
| | * | | | | | | | Add installer | David Heinemeier Hansson | 2018-12-13 | 3 | -9/+17 | |
| | | | | | | | | | ||||||
| | * | | | | | | | Tidy up and example testing | David Heinemeier Hansson | 2018-12-13 | 1 | -3/+27 | |
| | | | | | | | | | ||||||
| | * | | | | | | | Simpler | David Heinemeier Hansson | 2018-12-13 | 1 | -3/+1 | |
| | | | | | | | | | ||||||
| | * | | | | | | | Document the ingress controllers | George Claghorn | 2018-12-13 | 6 | -1/+168 | |
| | | | | | | | | | ||||||
| | * | | | | | | | Document ingress configuration | George Claghorn | 2018-12-13 | 1 | -6/+147 | |
| | | | | | | | | | ||||||
| | * | | | | | | | Need to explain testing too | David Heinemeier Hansson | 2018-12-12 | 1 | -0/+5 | |
| | | | | | | | | | ||||||
| | * | | | | | | | Explain all test helpers | David Heinemeier Hansson | 2018-12-12 | 1 | -1/+12 | |
| | | | | | | | | | ||||||
| | * | | | | | | | Stating the obvious | David Heinemeier Hansson | 2018-12-12 | 1 | -0/+1 | |
| | | | | | | | | | ||||||
| | * | | | | | | | Routing documentation | David Heinemeier Hansson | 2018-12-12 | 4 | -0/+60 | |
| | | | | | | | | | ||||||
| | * | | | | | | | Consistent naming on all factory methods | David Heinemeier Hansson | 2018-12-12 | 2 | -4/+8 | |
| | | | | | | | | | ||||||
| | * | | | | | | | Basic docs for most classes | David Heinemeier Hansson | 2018-12-12 | 8 | -0/+52 | |
| | | | | | | | | | ||||||
| | * | | | | | | | Note about incineration | David Heinemeier Hansson | 2018-12-12 | 1 | -0/+6 | |
| | | | | | | | | | ||||||
| | * | | | | | | | Base note | David Heinemeier Hansson | 2018-12-12 | 1 | -0/+1 | |
| | | | | | | | | | ||||||
| | * | | | | | | | Describe the conductor controller | David Heinemeier Hansson | 2018-12-12 | 1 | -0/+11 | |
| | | | | | | | | | ||||||
| | * | | | | | | | George ended up doing a lot of the actual code | David Heinemeier Hansson | 2018-12-12 | 1 | -3/+3 | |
| | | | | | | | | | ||||||
| | * | | | | | | | Basic description of the framework | David Heinemeier Hansson | 2018-12-12 | 1 | -1/+91 | |
| | | | | | | | | | ||||||
| | * | | | | | | | Address objects are value objects | David Heinemeier Hansson | 2018-12-06 | 2 | -0/+14 | |
| | | | | | | | | | ||||||
| | * | | | | | | | Add address wrapping | David Heinemeier Hansson | 2018-12-06 | 2 | -0/+16 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Makes it easier to deal with addresses that may already have been converted | |||||
| | * | | | | | | | Add a :all catch-all route | David Heinemeier Hansson | 2018-12-06 | 2 | -2/+21 | |
| | | | | | | | | | ||||||
| | * | | | | | | | Include the gem version in the user agent string | George Claghorn | 2018-12-04 | 2 | -2/+3 | |
| | | | | | | | | |