 bac4b78e09
			
		
	
	
	bac4b78e09
	
	
	
		
			
			* Possibility to not use whitelist but allow anyone with write access * fix existing test * rename migration function * Try to give a better name for migration step * Clear settings if higher level setting is not set * Move official reviews to db instead of counting approvals each time * migration * fix * fix migration * fix migration * Remove NOT NULL from EnableWhitelist as migration isn't possible * Fix migration, reviews are connected to issues. * Fix SQL query issues in GetReviewersByPullID. * Simplify function GetReviewersByIssueID * Handle reviewers that has been deleted * Ensure reviews for test is in a well defined order * Only clear and set official reviews when it is an approve or reject.
		
			
				
	
	
		
			81 lines
		
	
	
		
			No EOL
		
	
	
		
			1.3 KiB
		
	
	
	
		
			YAML
		
	
	
	
	
	
			
		
		
	
	
			81 lines
		
	
	
		
			No EOL
		
	
	
		
			1.3 KiB
		
	
	
	
		
			YAML
		
	
	
	
	
	
| -
 | |
|   id: 1
 | |
|   type: 1
 | |
|   reviewer_id: 1
 | |
|   issue_id: 2
 | |
|   content: "Demo Review"
 | |
|   updated_unix: 946684810
 | |
|   created_unix: 946684810
 | |
| -
 | |
|   id: 2
 | |
|   type: 1
 | |
|   reviewer_id: 534543
 | |
|   issue_id: 534543
 | |
|   content: "Invalid Review #1"
 | |
|   updated_unix: 946684810
 | |
|   created_unix: 946684810
 | |
| -
 | |
|   id: 3
 | |
|   type: 1
 | |
|   reviewer_id: 1
 | |
|   issue_id: 343545
 | |
|   content: "Invalid Review #2"
 | |
|   updated_unix: 946684810
 | |
|   created_unix: 946684810
 | |
| -
 | |
|   id: 4
 | |
|   type: 0 # Pending review
 | |
|   reviewer_id: 1
 | |
|   issue_id: 2
 | |
|   content: "Pending Review"
 | |
|   updated_unix: 946684810
 | |
|   created_unix: 946684810
 | |
| -
 | |
|   id: 5
 | |
|   type: 2
 | |
|   reviewer_id: 1
 | |
|   issue_id: 3
 | |
|   content: "New review 1"
 | |
|   updated_unix: 946684810
 | |
|   created_unix: 946684810
 | |
| -
 | |
|   id: 6
 | |
|   type: 0
 | |
|   reviewer_id: 2
 | |
|   issue_id: 3
 | |
|   content: "New review 3"
 | |
|   updated_unix: 946684811
 | |
|   created_unix: 946684811
 | |
| -
 | |
|   id: 7
 | |
|   type: 3
 | |
|   reviewer_id: 3
 | |
|   issue_id: 3
 | |
|   content: "New review 4"
 | |
|   updated_unix: 946684812
 | |
|   created_unix: 946684812
 | |
| -
 | |
|   id: 8
 | |
|   type: 1
 | |
|   reviewer_id: 4
 | |
|   issue_id: 3
 | |
|   content: "New review 5"
 | |
|   updated_unix: 946684813
 | |
|   created_unix: 946684813
 | |
| -
 | |
|   id: 9
 | |
|   type: 3
 | |
|   reviewer_id: 2
 | |
|   issue_id: 3
 | |
|   content: "New review 3 rejected"
 | |
|   updated_unix: 946684814
 | |
|   created_unix: 946684814
 | |
| 
 | |
| -
 | |
|   id: 10
 | |
|   type: 3
 | |
|   reviewer_id: 100
 | |
|   issue_id: 3
 | |
|   content: "a deleted user's review"
 | |
|   updated_unix: 946684815
 | |
|   created_unix: 946684815 |