Fix #85 manipulate session[:shopify] as YAML

[choonkeat]

instead of as ShopifyAPI::Session object (which session cannot handle properly)

[csaunders]

This isn't a great solution. Instead of storing the entire session we should be coming up with a better way of patching this.

I was thinking that just having a shops table would work though that does add the requirement for a database to be required in order to just play with the app. Alternatively could use something like a ShopsRepository that does something like stores the information in a global variable or something.

This in-memory approach wouldn't really work for a production setup but you could switch out the repo backend to just be active record.

session[:id] = ShopsRepo.store(shopify_session)
# 17def012-9d4a-4982-8d04-87abe379dadd

shopify_session = ShopsRepo.fetch(session[:id])

class ShopsRepo
  def self.store(session)
    id = SecureRandom.uuid
    repo[id] = {token: session.token, url: session.url} # or simply repo[id] = session
    id
  end

  def self.fetch(id)
    if data = repo[id]
      ShopifyAPI::Session.new(token: data[:token], url: data[:url])
    end
  end

  def self.repo
    @@repo ||= {}
  end
end

[csaunders]

See #92