adding multiple routing using a single path

[codad5]

Just like express.js router this is to add multiple routing methods for a single route path
just like

#multiple route 
$router->route('/hello')
    ->get(function($req, $res) {
        return $res->send("<h1>Hello World</h1>")->status(200); // sends html response
    })
    ->post(function($req, $res) {
        return $res->send([
            "message" => "Hello World posted"
        ])->status(200); // sends json response
    });

instead of

# GET route
$router->get("/hello", function($req, $res) {
    return $res->send("<h1>Hello World</h1>")->status(200); // sends html response
});

# POST route
$router->post("/hello", function($req, $res) {
   return $res->send([
            "message" => "Hello World posted"
        ])->status(200); // sends json response
});

[aosasona]

The src folder is not where the view is at, this assumes they have autoloader mapped to /src too and the index.php is outside that, the initial version was written to match the examples folder. Check it out for reference.

[codad5]

Ok

[aosasona]

Having null as a route is not user-friendly or visually pleasing, perhaps you could change it to simply take empty strings?

[codad5]

Yes . You are right. I notice that after creating the pull request.. sorry my bad

[aosasona]

Ah, great you caught this 😅

[codad5]

😂 that was the first issue I ran into

[aosasona]

Oh it’s alright. Also could you for the addition to the readme, could you switch to double quotes instead just to make it consistent? On 1 Aug 2022, at 01:27, Aniezeofor Chibueze Michael ***@***.***> wrote: @codad5 commented on this pull request. In src/Router.php:

@@ -123,23 +124,39 @@ private function add_route($path, $cb, $method = "GET")

} # Create a GET route - public function get($path, $cb) { + public function get($path = null, $cb) { Yes . You are right. I notice that after creating the pull request.. sorry my bad —Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you commented.Message ID: ***@***.***>

[codad5]

Had to give reference to the main route object in the sub route .. due to the $route->serve() method

[aosasona]

For dev testing using using the PHP server, this is not necessary and as I mentioned, it is all routed to /examples, you are redirecting all traffic here and it might try to look for /examples in /examples itself again.