|
From: | Tatiana Sholokhova |
Subject: | Re: Web interface review |
Date: | Sun, 22 Jul 2018 16:57:18 +0200 |
Hello Tatiana!
Tatiana Sholokhova <address@hidden> writes:
> Hello Clément!
>
> Thank you for your review!
>
> I fixed most of the problems you noticed and rebased commits as you advised
> .
>
> I couldn't fix the problem with several calling of (with-critical-section).
> As I wrote to IRC channel, I tried to put '(with-critical-section
> db-channel (db)' around '(let* ...)' and I received an error:
> ```
> In web/server.scm:
> 279:25 0 (_)
> Throw to key `vm-error' with args `(vm-run "Too few values returned to
> continuation" ())'.
> ```
It's because 'respond-html' returns several values. I think you could
do:
(respond-html
(with-critical-section ...
(let* ...)))
> Could you give a status about the pagination?
>>
> Pagination works correctly with evaluations, but it doesn't work correctly
> with builds. In some cases, we have builds missing. It happens due to equal
> timestamp values, so we need to filter build by (timestamp, id) tuple key.
>
> What else do we need to do before the merge?
Once we have something consistent, we can push. And we can add stuff
afterwards of course. Do you think it would be feasible to fix the
pagination before the merge?
I won't have time to look at your update before tomorrow night, I'll let
you know then!
Thanks,
Clément
[Prev in Thread] | Current Thread | [Next in Thread] |