Do not consider moved accounts as non-functional
This might be somewhat controversial, but putting a redirection up makes the account unusable, which works against use cases such as using the moved account as backup in case the new one temporarily fails (sure, the redirection can be temporarily removed, but it is a hassle)
This commit is contained in:
		
							parent
							
								
									2448568d34
								
							
						
					
					
						commit
						b25665d2e8
					
				@ -169,7 +169,7 @@ class User < ApplicationRecord
 | 
				
			|||||||
  end
 | 
					  end
 | 
				
			||||||
 | 
					
 | 
				
			||||||
  def functional?
 | 
					  def functional?
 | 
				
			||||||
    confirmed? && approved? && !disabled? && !account.suspended? && account.moved_to_account_id.nil?
 | 
					    confirmed? && approved? && !disabled? && !account.suspended?
 | 
				
			||||||
  end
 | 
					  end
 | 
				
			||||||
 | 
					
 | 
				
			||||||
  def unconfirmed_or_pending?
 | 
					  def unconfirmed_or_pending?
 | 
				
			||||||
 | 
				
			|||||||
		Loading…
	
	
			
			x
			
			
		
	
		Reference in New Issue
	
	Block a user