* 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 |