Skip to content

Commit

Permalink
Merge pull request #1165 from alphagov/extend-scheduled-publishing-de…
Browse files Browse the repository at this point in the history
…lay-seconds-to-bigint

fix Integer field overflow in JsonImporter, to handle extreme values
  • Loading branch information
aldavidson authored Oct 16, 2023
2 parents 8c0c135 + e9c4523 commit d1e4c97
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 2 deletions.
Original file line number Diff line number Diff line change
@@ -0,0 +1,9 @@
class ChangeScheduledPublishingDelaySecondsToBigint < ActiveRecord::Migration[7.0]
def up
change_column :content_items, :scheduled_publishing_delay_seconds, :bigint
end

def down
change_column :content_items, :scheduled_publishing_delay_seconds, :integer
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 d1e4c97

Please sign in to comment.