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 -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
