HTTP examples don't work about rom-rb.org HOT 6 CLOSED

@flash-gordon ahh yeah we talked about this recently. Yeah JSON::Object makes sense, and the error messages should be changed in 2.0.0s 🙂 anyhow, not strictly related to this issue here, we can follow-up in a discourse thread (I'll create it later).

@UsAndRufus could you open a PR with the example updated like in my comment above?

from rom-rb.org.

@UsAndRufus oh it's this one

from rom-rb.org.

Thanks, for now let's do this:

module GitHub
  module Resources
    class Organizations < ROM::Relation[:http]
      schema(:orgs) do
        attribute :id, Types::Integer
        attribute :name, Types::String
        attribute :created_at, Types::JSON::Time
        attribute :updated_at, Types::JSON::Time
      end

      def by_name(name)
        append_path(name)
      end
    end
  end
end

in the future, we probably want to add Types::JSON::(String,Integer) even though they'd be plain definitions, but it's a weird "gotcha" that is clearly problematic. We already did this with Types::Params::String that originally wasn't present. /cc @flash-gordon makes sense to you?

from rom-rb.org.

it does. Not sure about arrays or object. Mb JSON::Object and alias for hash? This can have implications for dry-schema though (that error messaging thing again)...

from rom-rb.org.

@solnic sure, which is the correct file to update? I can see it in the rom-rb.org repo and the rom-http repo. Is there a Rake task or something that copies them across or will both need updating manually?

from rom-rb.org.

Closing this as PR that addresses it is merged :)

from rom-rb.org.