Route handling about kakapo.js

[rpunkfu]

I think we should support queries! :) Also we probably should pass object as a second argument to filter db.find

import { router } from 'kakapo';

router.get('/users?size=42&page=7', usersHandler);
router.get('/users/:id', usersHandler);

const usersHandler = (request, database) => ({
    users: database.find('user', request.params, request.query.size);
});

[rpunkfu]

As I think of that, we shouldn't have router & database being so tightly coupled... imo router shouldn't know anything about database:

import { Database, Router } from 'kakapo';

const database = new Database(/* bla bla bla */);
const router = new Router(/* bla bla bla */);

database.register('user', /* bla bla bla */);

router.get('/users?size=42&page=7', usersHandler);
router.get('/users/:id', usersHandler);

const usersHandler = request => ({
    users: database.find('user', {id: request.params.id}, {size: request.query.size});
});

[zzarcon]

request.params already implemented 💪 and I like the request.query i was not sure about which name to use for that one ^^

[zzarcon]

@oskarcieslik should we close this issue and create a new one with the point of decoupling the router and the db?

[rpunkfu]

Yes, I will close this after tests for query implementation pass :)