aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Action Text: bundle package.json in built gem.Kasper Timm Hansen2019-01-181-1/+1
| | | | | | | | | | | | | | | | | | After 866da19fd9fc12d001ab99ac87890ea1cebb2cd9 we now use package.json to install Action Text's JS dependencies (see JS_PACKAGE_PATH). But when the gem pacakge for Action Text is built, package.json was not included so running `rails action_text:install` in a Rails app would fail with: ``` rails action_text:install rails aborted! Errno::ENOENT: No such file or directory @ rb_sysopen - xxx/gems/actiontext-6.0.0.alpha/lib/templates/../../package.json ./bin/rails:4:in `<main>' Tasks: TOP => app:template (See full trace by running task with --trace) ```
* Use released webpackerRafael Mendonça França2019-01-173-12/+7
|
* Merge pull request #34966 from ↵Rafael Mendonça França2019-01-173-4/+14
|\ | | | | | | | | | | bogdanvlviv/ensure-ar-relation-exists-allows-permitted-params Ensure that AR::Relation#exists? allows only permitted params
| * Ensure that AR::Relation#exists? allows only permitted paramsbogdanvlviv2019-01-173-7/+17
| | | | | | | | | | Clarify changelog entry Related to #34891
* | Merge pull request #34954 from rails/rm-remove-5.2-deprecationsRafael França2019-01-1767-1189/+210
|\ \ | | | | | | Remove all code deprecated in Rails 5.2
| * | Remove deprecated `after_bundle` helper inside plugins templatesRafael Mendonça França2019-01-175-53/+16
| | |
| * | Remove deprecated `server` argument from the rails server commandRafael Mendonça França2019-01-173-25/+6
| | |
| * | Remove deprecated support to old `config.ru` that use the application class ↵Rafael Mendonça França2019-01-173-27/+4
| | | | | | | | | | | | as argument of `run`
| * | Remove deprecated `environment` argument from the rails commandsRafael Mendonça França2019-01-175-55/+10
| | |
| * | Remove deprecated `capify!`Rafael Mendonça França2019-01-173-18/+4
| | |
| * | Remove secret_token rack env and cookie upgrade codeRafael Mendonça França2019-01-177-261/+19
| | | | | | | | | | | | Now that secret_token was removed all this code is now dead.
| * | Remove deprecated `config.secret_token`Rafael Mendonça França2019-01-177-233/+13
| | |
| * | Remove deprecated `Module#reachable?` methodRafael Mendonça França2019-01-174-58/+5
| | |
| * | Remove deprecated `#acronym_regex` method from `Inflections`Rafael Mendonça França2019-01-174-9/+5
| | |
| * | Remove deprecated `#set_state` from the transaction objectRafael Mendonça França2019-01-173-29/+4
| | |
| * | Remove deprecated `#supports_statement_cache?` from the database adaptersRafael Mendonça França2019-01-173-11/+4
| | |
| * | Remove deprecated `#insert_fixtures` from the database adaptersRafael Mendonça França2019-01-175-43/+5
| | |
| * | Remove deprecated ↵Rafael Mendonça França2019-01-173-9/+4
| | | | | | | | | | | | `ActiveRecord::ConnectionAdapters::SQLite3Adapter#valid_alter_table_type?`
| * | Do not allow passing the column name to `sum` when a block is passedRafael Mendonça França2019-01-173-9/+11
| | |
| * | Do not allow passing the column name to `count` when a block is passedRafael Mendonça França2019-01-173-9/+11
| | |
| * | Remove delegation of missing methods in a relation to arelRafael Mendonça França2019-01-173-20/+4
| | |
| * | Remove delegation of missing methods in a relation to private methods of the ↵Rafael Mendonça França2019-01-173-12/+4
| | | | | | | | | | | | class
| * | Change `SQLite3Adapter` to always represent boolean values as integersRafael Mendonça França2019-01-178-109/+44
| | |
| * | Remove ability to specify a timestamp name for `#cache_key`Rafael Mendonça França2019-01-174-31/+10
| | |
| * | Remove deprecated `ActiveRecord::Migrator.migrations_path=`Rafael Mendonça França2019-01-173-17/+4
| | |
| * | Remove deprecated `expand_hash_conditions_for_aggregates`Rafael Mendonça França2019-01-173-46/+7
| | |
| * | Remove deprecated image_alt helperRafael Mendonça França2019-01-173-43/+4
| | |
| * | Remove deprecated `fragment_cache_key` helper in favor of ↵Rafael Mendonça França2019-01-173-27/+4
| | | | | | | | | | | | `combined_fragment_cache_key`
| * | Remove deprecated methods in ActionDispatch::TestResponseRafael Mendonça França2019-01-173-34/+7
| | | | | | | | | | | | | | | `#success?`, `missing?` and `error?` were deprecated in Rails 5.2 in favor of `#successful?`, `not_found?` and `server_error?`.
| * | Update the promisse that ActionController::TestCase will be extractedRafael Mendonça França2019-01-171-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | We are past 5.1 and it was not extrated yet, so while we still have plans they will not be realized on 6.0, so it is better to not set expectations of which release will exclude it just yet. [ci skip]
* | | Merge pull request #34967 from lifo/am-handle-duplicate-emailsGeorge Claghorn2019-01-174-10/+9
|\ \ \ | |/ / |/| | Update test schema.rb to reflect the recent changes
| * | Use create_and_extract_message_id! to create an inbound email.Pratik Naik2019-01-171-2/+1
| | | | | | | | | | | | This makes sure the created email has checksum and message id columns set.
| * | Specify a name for [ message_id, message_checksum ] index to ensure the name ↵Pratik Naik2019-01-173-8/+8
| | | | | | | | | | | | | | | | | | does not exceed the limit. Also, update test schema to reflect the recent changes.
* | | Merge pull request #34412 from gmcgibbon/redirect_to_allow_other_hostRafael França2019-01-177-29/+87
|\ \ \ | | | | | | | | Ensure external redirects are explicitly allowed
| * | | Ensure external redirects are explicitly allowedGannon McGibbon2019-01-177-29/+87
| | | | | | | | | | | | | | | | Add `fallback_location` and `allow_other_host` options to `redirect_to`.
* | | | Merge pull request #34962 from lifo/am-handle-duplicate-emailsGeorge Claghorn2019-01-176-15/+44
|\ \ \ \ | | |/ / | |/| | Ensure Action Mailbox processes an email only once
| * | | Ensure Action Mailbox processes an email only once when received multiple timesPratik Naik2019-01-176-15/+44
| | |/ | |/| | | | | | | | | | | | | | | | This also adds a new column, message_checksum, to the action_mailbox_inbound_emails table for storing SHA1 digest of the email source. Additionally, it makes generating the missing message id deterministic and adds a unique index on message_checksum and message_id to detect duplicate emails.
* | | Merge pull request #34958 from javan/actiontext/trix-as-peerJavan Makhmali2019-01-173-11/+23
|\ \ \ | |/ / |/| | Declare Trix as peer dependency of Action Text's npm package
| * | Tidy up action_text:install taskJavan Makhmali2019-01-171-9/+20
| | | | | | | | | | | | Automate installing the appropriate packages with yarn and appending them to the default application.js pack.
| * | Make trix a peer dependency of actiontext since it’s not used directlyJavan Makhmali2019-01-172-2/+3
| | |
* | | Merge pull request #34953 from gmcgibbon/seed_with_inline_jobsRafael França2019-01-173-1/+29
|\ \ \ | |/ / |/| | Seed database with inline ActiveJob job adapter
| * | Seed database with inline ActiveJob job adapterGannon McGibbon2019-01-173-1/+29
| | |
* | | Merge pull request #34213 from matildasmeds/guides_session_guidelines_2Rafael França2019-01-171-68/+31
|\ \ \ | |_|/ |/| | Edit Security Guide's Session Guidelines & Custom Credentials [skip ci]
| * | Merge branch 'master' into guides_session_guidelines_2Matilda Smeds2018-12-09473-9721/+14396
| |\ \
| * | | Update guides/source/security.mdDerek Prior2018-10-181-1/+1
| | | |
| * | | Update guides/source/security.mdDerek Prior2018-10-181-1/+1
| | | |
| * | | Edit Security Guide's Session Guidelines & Custom Credentials [skip ci]Matilda Smeds2018-10-141-68/+31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | * Edit Session Guidelines to achieve tighter prose and accuracy * Remove mentions related to earlier Rails versions * Add links to ActionController guide and Custom Credentials part * Clarify Custom Credentials part
* | | | Merge pull request #33419 from bogdanvlviv/update-active_storageGeorge Claghorn2019-01-166-1/+31
|\ \ \ \ | | | | | | | | | | `rake app:update` should update active_storage
| * | | | Add foreign key to active_storage_attachments for `blob_id` via new migrationbogdanvlviv2019-01-166-1/+31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We need this in order to be able to add this migration for users that use ActiveStorage during update their apps from Rails 5.2 to Rails 6.0. Related to #33405 `rake app:update` should update active_storage `rake app:update` should execute `rake active_storage:update` if it is used in the app that is being updated. It will add new active_storage's migrations to users' apps during update Rails. Context https://github.com/rails/rails/pull/33405#discussion_r204239399 Also, see a related discussion in the Campfire: https://3.basecamp.com/3076981/buckets/24956/chats/12416418@1236713081
* | | | | Add a space in framework names. Matches Active Record in generated Gemfile.Kasper Timm Hansen2019-01-161-2/+2
| | | | |