Skip to content

Commit

Permalink
fix Integer field overflow in JsonImporter, to handle extreme values …
Browse files Browse the repository at this point in the history
…coming through from Whitehall via MongoDB
  • Loading branch information
aldavidson committed Oct 16, 2023
1 parent 8c0c135 commit dadcc8d
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 2 deletions.
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
class ChangeScheduledPublishingDelaySecondsToBigint < ActiveRecord::Migration[7.0]
def change
change_column :content_items, :scheduled_publishing_delay_seconds, :bigint

Check failure on line 3 in db/migrate/20231016112610_change_scheduled_publishing_delay_seconds_to_bigint.rb

View workflow job for this annotation

GitHub Actions / Lint Ruby / Run RuboCop

Rails/ReversibleMigration: change_column is not reversible. (https://rails.rubystyle.guide#reversible-migration, https://api.rubyonrails.org/classes/ActiveRecord/Migration/CommandRecorder.html)
end
end
4 changes: 2 additions & 2 deletions db/schema.rb
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@
#
# It's strongly recommended that you check this file into your version control system.

ActiveRecord::Schema[7.0].define(version: 2023_08_30_093643) do
ActiveRecord::Schema[7.0].define(version: 2023_10_16_112610) do
# These are extensions that must be enabled in order to support this database
enable_extension "pgcrypto"
enable_extension "plpgsql"
Expand All @@ -35,7 +35,7 @@
t.datetime "first_published_at"
t.datetime "public_updated_at"
t.datetime "publishing_scheduled_at"
t.integer "scheduled_publishing_delay_seconds"
t.bigint "scheduled_publishing_delay_seconds"
t.jsonb "details", default: {}
t.string "publishing_app"
t.string "rendering_app"
Expand Down

0 comments on commit dadcc8d

Please sign in to comment.