Top Level Reviews API Endpoint

Review Request #12028 — Created Feb. 3, 2022 and discarded — Latest diff uploaded

Information

Review Board
master

Reviewers

  • Added top level reviews endpoint.
  • Ability to filter by username using ?user=username parameter.
  • Ability to filter by repository name using ?repository=repoName parameter.
  • Ability to filter by date range using ?last-updated-from and ?last-updated-to parameters.
  • Ability to filter by review group using ?review-group=group_name parameter.
  • Added tests for the new root_reviews resource.
  • Added documentation for top level reviews endpoint.
  • Wrote unit tests for all new functionality
  • Ran unit tests for root_reviews resource and ensure they pass
  • Tested new API endpoint manually and confirmed results were as expected.

Diff Revision 6

This is not the most recent revision of the diff. The latest diff is revision 11. See what's changed.

orig
1
2
3
4
5
6
7
8
9
10
11

Commits

First Last Summary ID Author
attempted adding root_review
4a98ef5189063d4f605a70f87cd78c696ead9e5d Taylor Christie
got endpoint working (rudementarily), added some basic tests
206769261bfca454dc42dca506e10bb67bb6f20f Taylor Christie
get tests working
62f5caf5437416438fe386d3169a3d2c95de44c7 Taylor Christie
Added user filtering by username. Added repository filtering by repository na...
64e37475a38cd7180e70065a0b85c0557b7726bd Taylor Christie
added date filtering, added review group filtering, added blank unit tests
ef5f0394cfbcf904b9ecffee4c01530494e1db62 Taylor Christie
removed unneeded imports
d01ead0c5789f6f273dde85ffc40820b514e2f1f Taylor Christie
added unit tests for last-updated-from and laste-updated-to parameters and ad...
46ef7bf956bd684356ad088143acc16ca8d53ca7 Taylor Christie
finished review-group parameter, unit tests, and flake8 formatting
a82f040f7ad92cc37ac053662d0d7380c8d551b7 Taylor Christie
added documentation
55c5247da821df4fb376238c31ac7a6a8f806f32 Taylor Christie
This diff has been split across 40 pages: 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 >
AUTHORS
Revision f420b77db7c838b5810855fd31f7a7d9297b2eec New Change
17 lines
18
	* Alexander Solovets
18
	* Alexander Solovets
19
	* Alexey Morozov
19
	* Alexey Morozov
20
	* Alexey Neyman
20
	* Alexey Neyman
21
	* Allyshia Sewdat
21
	* Allyshia Sewdat
22
	* Amir Yalon
22
	* Amir Yalon
23
	* Anahita Mohapatra

   
24
	* André Klitzing
23
	* André Klitzing
25
	* Andrew Hills
24
	* Andrew Hills
26
	* Andrew Hong
25
	* Andrew Hong
27
	* Andrey Fedorov
26
	* Andrey Fedorov
28
	* Anni Cao
27
	* Anni Cao
39 lines
68
	* Corey Hickey
67
	* Corey Hickey
69
	* Craig Rodrigues
68
	* Craig Rodrigues
70
	* Cristian Messel
69
	* Cristian Messel
71
	* cstejerean
70
	* cstejerean
72
	* Curtis Miller
71
	* Curtis Miller
73
	* Dalton Ronan

   
74
	* Damian Johnson
72
	* Damian Johnson
75
	* Dan Bentley
73
	* Dan Bentley
76
	* Dan LaMotte
74
	* Dan LaMotte
77
	* Dan Minor
75
	* Dan Minor
78
	* Dan Savilonis
76
	* Dan Savilonis
94 lines
173
	* Kay Röpke
171
	* Kay Röpke
174
	* Kevin Chiu
172
	* Kevin Chiu
175
	* Kevin Kubasik
173
	* Kevin Kubasik
176
	* Kimon Papahadjopoulos
174
	* Kimon Papahadjopoulos
177
	* Kristina Vandergulik
175
	* Kristina Vandergulik
178
	* Kyle McLean

   
179
	* Laila Agaev
176
	* Laila Agaev
180
	* Lance Chen
177
	* Lance Chen
181
	* Lance Hudson
178
	* Lance Hudson
182
	* Laurent Nicolas
179
	* Laurent Nicolas
183
	* Leo Soto
180
	* Leo Soto
160 lines
conftest.py
dev-requirements.txt
setup.cfg
setup.py
contrib/docker/build.py
contrib/internal/prepare-dev.py
contrib/internal/webapi-lint.py
docs/manual/_ext/webapidocs.py
docs/manual/admin/sites/search-indexing.rst
docs/manual/extending/extensions/hooks/integration-hook.rst
docs/manual/fixtures/initial_data.json
docs/manual/webapi/2.0/resources/index.rst
docs/manual/webapi/2.0/resources/root-review.rst
docs/releasenotes/4.0.5.rst
docs/releasenotes/4.0.6.rst
docs/releasenotes/conf.py
docs/releasenotes/index.rst
reviewboard/__init__.py
reviewboard/dependencies.py
This diff has been split across 40 pages: 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 >
Loading...