Hi,
Thanks for the excellent work on the new commitfest app. It looks
awesome so far, though I'm betting the commitfest manager is the one who
reaps the most benefits.
Wanted to highlight this request:
Andres Freund wrote:
> What I'm also missing from the old app is that previously 'reviews'
> could explicitly be linked from the app. Now there's a list of emails in
> the thread, nice!, but in bigger threads that really doesn't help to
> find the actual review.
Note for instance the "BRIN inclusion operator class" patch here,
https://commitfest.postgresql.org/3/31/
The link points to the generic BRIN thread I started (you can see it in
the history). If you list all attachments you can see that all the BRIN
patches are linked; Emre's patch is there, it seems, only because it's
the one most recently posted.
Not sure what to do here, but it's somewhat annoying.
Also, I was pretty used to offline operation with the old one: I could
load the page, for instance
https://commitfest-old.postgresql.org/action/commitfest_view?id=25
and the "patch" links alongside each patch had the message-ids with
which I could search my local mbox. In the new one, the only way I can
get the message-id is by opening the patch page. It would be pretty
useful to have a "copy message-id to clipboard" button, or something
similar, so that I could transfer operation from the web browser to my
mail client. Having one message-id per patch in the commitfest summary
page is enough for my use case (probably pointing to the most recent
attachment in the linked threads.)
--
Álvaro Herrera http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services