Skip to content
This repository was archived by the owner on Oct 19, 2018. It is now read-only.

During the install it should install a migration that created the needed tables #24

Open

Description

@johansmitsnl

Now when you migrate the database you loose the injected schema.
And missing the tables: hyperloop_connections and hyperloop_queued_messages

It should install these 2 migrations:

class InstallHyperloopConnections < ActiveRecord::Migration[5.2]
  def change

    create_table :hyperloop_connections do |t|
      t.string :channel
      t.string :session
      t.datetime :created_at
      t.datetime :expires_at, index: true
      t.datetime :refresh_at
    end


    create_table :hyperloop_queued_messages do |t|
      t.integer :connection_id, index: true
      t.text :data
    end
  end
end

Should there be an index on the channel of session?

Activity

  1. added this to the Release 0.15 milestone on Aug 7, 2018
  2. catmando commented on Aug 7, 2018

    @catmando
    Contributor

    Yes Indeed! This will be done when we transition to the new new Hyperstack architecture.

  3. johansmitsnl commented on Aug 7, 2018

    @johansmitsnl
    ContributorAuthor

    Also a question about the way of installing. Why is this a generate job instead of a normal task like active_storage?

    rails active_storage:install             # Copy over the migration needed to the application
    rails active_storage:install:migrations  # Copy migrations from active_storage to application
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions