Lighthouse has a new layout. Prefer the old one? Return to the old layout, and switch back any time from the link at the top of each page.

This project is archived and is in readonly mode.

[Patch] Metals should be loaded from Engines

#2045

I'd like to bundle a simple "/backlog" request to my RudeQueue plugin (http://github.com/matthewrudy/ru...

and I'd like to do it as a metal


class RudeQueue::Backlog
  def self.call(env)
    if env["PATH_INFO"] =~ /^\/backlog/
      [200, {"Content-Type" => "text/html"}, [RudeQueue.backlog.to_s]]
    else
      [404, {"Content-Type" => "text/html"}, ["Not Found"]]
    end
  end
end

it seems to make sense,

however Engines don't currently load their Metal.

Here's my fork which fixes that http://github.com/matthewrudy/ru...

and a patch is also attached.

Any suggestions on how to better format the code, or manage the paths is very much welcome. (notably it seems to be very hard to test in code... I've tested it manually)

Reported by MatthewRudy · February 22nd, 2009 @ 08:18 PM

State: committed
Milestone: 2.x
Assigned to: josh josh
Importance: none

Activity

  1. DHH
    DHH
    • Assigned user set to josh

    February 27th, 2009 @ 01:33 PM

  2. Repository
    Repository
    • State changed from new to committed

    (from [319106d09c0ec2daf8b5345f525f1c97b6368ce2]) Metal can now line in plugins under app/metal [#2045 [Patch] Metals should be loaded from Engines state:committed]

    Signed-off-by: David Heinemeier Hansson david@loudthinking.com http://github.com/rails/rails/co...

    February 27th, 2009 @ 01:43 PM