sympy/sympy-bot — Issue #41: web app improvements
Author: certik
Created: 2011-10-12 06:12:03+00:00
State: open

A few suggestions:

On the main page I would add a title of every pull request, instead of just a number.

"Repository and branch: https://github.com/smichr/sympy polyx": I would make both repo and branch links (now only branch is a link).

sort the pull request by the "creation date" on the front page, and also split them to "can be merged", "can't be merged".

I think that the front page should emulate this page: https://github.com/sympy/sympy/pulls, just the part about pull requests. With all the info, just do it better and more usable. So there would be buttons like "mergeable", "open", "close", "recent", "sort by contributors 'newness' to sympy" and so on. And we can make it really usable and put a big link to sympy.org for it, so that people start reviewing things more.

It would be awesome if we could have all the important metadata in the webapp, and then even a search function there, which is something that is annoyingly missing from GitHub's interface. Indeed, if it were more or less like the GitHub page (though I could think of a few improvements :-), that would be great.

It would also be cool if it were full of html 5, though I realize that that's way more work :)

Could we also add "by whomever" (i.e., the person who did the review)?
Yes, also together with the picture and put everything into a nice table. And add buttons to sort by various criteria.

fix json (use try/except)

I found another bug:

http://reviews.sympy.org/report/agZzeW1weTNyCwsSBFRhc2sYiHUM

if I upload a test result, before the app updates the list of pull requests from github, the pull request number is missing. Internally, if I can't find the pull request in the database, I should create the pull request, with the given number anyway (in the database). The update will then just update the data about it, so all will work.

The pull requests currently remain on the front page forever. I'll implement hiding the closed pull requests.