Merge lp:~vhata/ibid/feeds-747635 into lp:ibid
Proposed by
Stefano Rivera
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Stefano Rivera | ||||
Approved revision: | 1029 | ||||
Merged at revision: | 1038 | ||||
Proposed branch: | lp:~vhata/ibid/feeds-747635 | ||||
Merge into: | lp:ibid | ||||
Diff against target: |
72 lines (+32/-9) 1 file modified
ibid/plugins/feeds.py (+32/-9) |
||||
To merge this branch: | bzr merge lp:~vhata/ibid/feeds-747635 | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Keegan Carruthers-Smith | Approve | ||
Stefano Rivera | Approve | ||
Max Rabkin | Pending | ||
Jonathan Hitchcock | Pending | ||
Review via email: mp+80198@code.launchpad.net |
This proposal supersedes a proposal from 2011-10-24.
Commit message
Make feeds syntax more intuitive/
Description of the change
Adds the ability to specify 'headlines' or 'articles' when fetching a bunch (and fixes the syntax so that if you just say 'latest article' it only fetches one).
To post a comment you must log in.
We don't use "starting at" any where else, we use "from" or "#".
Should we not standardise a bit?
Otherwise I quite like it. My initial response was that this should have been fixed by changing article(), but this does seem more user-friendly.