'Load more' functionality for the action feed

Review Request #3260 — Created Aug. 3, 2012 and discarded

Information

Review Board

Reviewers

This patch is built on top of the one in r/3151/
Changes include:
   * generating the action feed by getting the action list via ajax. 
   * brand new "Load more" button

 
Description From Last Updated

I *think* you can just do: display_verb = property(get_verb_display) Might be nice to just add a simple comment saying that …

chipx86chipx86

Ideally, should be alphabetized. We're not always good at this, but for new code, yeah :)

chipx86chipx86

Really, this should be: changeitem .append(' changed from ') .append($('').text(oldValue) ... ...;

chipx86chipx86

One per line.

chipx86chipx86

The ) should be on the previous line.

chipx86chipx86

I think these could just go inline when building the div below.

chipx86chipx86

This should be more clear as to what we're loading. Should mention actions.

chipx86chipx86

Space before {

chipx86chipx86

Remove these blank lines.

chipx86chipx86

Probably want to remove that.

chipx86chipx86

. No end tag.

chipx86chipx86

Can we just get this from the review request we link to? I don't know that exposing this is right.

chipx86chipx86

How necessary is this? It'd be nice to use the linked resource instead.

chipx86chipx86

"change description"

chipx86chipx86

"Human-readable"

chipx86chipx86

Rather than basing it on a last ID like this, it might be better to allow searching by a time …

chipx86chipx86
chipx86
  1. 
      
  2. reviewboard/reviews/models.py (Diff revision 1)
     
     
     
     
    Show all issues
    I *think* you can just do:
    
    display_verb = property(get_verb_display)
    
    Might be nice to just add a simple comment saying that get_verb_display is provided automatically .by the verb field. I was confused at first as to where this came from.
    1. I tried writing it as you suggested, but got a syntax error. 
      Added the comment.
  3. reviewboard/static/rb/css/action-feed.less (Diff revision 1)
     
     
     
     
     
     
     
    Show all issues
    Ideally, should be alphabetized. We're not always good at this, but for new code, yeah :)
  4. reviewboard/static/rb/js/action-feed.js (Diff revision 1)
     
     
     
     
     
    Show all issues
    Really, this should be:
    
    changeitem
        .append(' changed from ')
        .append($('<i/>').text(oldValue)
        ...
        ...;
  5. reviewboard/static/rb/js/action-feed.js (Diff revision 1)
     
     
     
     
    Show all issues
    One per line.
  6. reviewboard/static/rb/js/action-feed.js (Diff revision 1)
     
     
     
    Show all issues
    The ) should be on the previous line.
  7. reviewboard/static/rb/js/action-feed.js (Diff revision 1)
     
     
     
     
     
    Show all issues
    I think these could just go inline when building the div below.
  8. reviewboard/static/rb/js/action-feed.js (Diff revision 1)
     
     
    Show all issues
    This should be more clear as to what we're loading. Should mention actions.
  9. reviewboard/static/rb/js/action-feed.js (Diff revision 1)
     
     
    Show all issues
    Space before {
  10. reviewboard/static/rb/js/datastore.js (Diff revision 1)
     
     
     
     
     
    Show all issues
    Remove these blank lines.
  11. reviewboard/static/rb/js/datastore.js (Diff revision 1)
     
     
    Show all issues
    Probably want to remove that.
  12. Show all issues
    <input .. />. No end tag.
  13. reviewboard/webapi/resources.py (Diff revision 1)
     
     
     
     
     
    Show all issues
    Can we just get this from the review request we link to? I don't know that exposing this is right.
    1. Doesn't the ReviewRequestResource send the actual id of the request and not the local-site one (in case there is a local site)?
      
  14. reviewboard/webapi/resources.py (Diff revision 1)
     
     
     
     
     
    Show all issues
    How necessary is this? It'd be nice to use the linked resource instead.
  15. reviewboard/webapi/resources.py (Diff revision 1)
     
     
    Show all issues
    "change description"
  16. reviewboard/webapi/resources.py (Diff revision 1)
     
     
    Show all issues
    "Human-readable"
  17. reviewboard/webapi/resources.py (Diff revision 1)
     
     
    Show all issues
    Rather than basing it on a last ID like this, it might be better to allow searching by a time range. Preferably, we shouldn't be passing around IDs much.
  18. 
      
BO
  1. All issues fixed in Review:3292.
  2. 
      
BO
Review request changed
Status:
Discarded