[Vortex] Make PushNotificationStatus a backed enum

Review Request #1087 — Created Jan. 3, 2024 and submitted

Information
pprkut
Lunr.Vortex
6771878...
Reviewers
lunr 
Make PushNotificationStatus a backed enum
GitHub actions run: https://github.com/pprkut/lunr.vortex/actions/runs/7942823291
  • 0
  • 0
  • 0
  • 1
  • 1
Description From Last Updated
smillernl
smillernl 
  1. 
  
    
 
    

 
    
  


    
 
    
 
      

  

 
    


    

 
    

    2. 







  2. 
 
    
  
  
    
  
   
    
    
    



    
 
    
  
    
   
   
    
    
     
     
     
     
     
    
    Show all issues
   
  
    
 
    

    



    
    So, why a backed enum? I don't see any value usage here.
    
  
    
 
    

 
    
  


    
 
    
 
      

  


    1. 
 
 
      
 
      
  
 
      
 
      
  
      In our projects, the value ends up in the database
      
 
      

      2. 


 
    


    

 
    

    3. 













  3. 
 
    
  
    
   
    
  
    
 
    

 
    
  


    
 
    
 
      

  

 
    


    

 
    

    4. 




  

 
















 





 

  

   




b.stoop




   

   







  

 



 

  


   

   

    
 
b.stoop


    
   


  

  

  



    




  1. 
 
    
  
    
   
    Ship It!
    
  
    
 
    

 
    
  


    
 
    
 
      

  

 
    


    

 
    

    2. 















  2. 
 
    
  
    
   
    
  
    
 
    

 
    
  


    
 
    
 
      

  

 
    


    

 
    

    3. 




  

 
















 



 

  

   




pprkut




   

   

  

 



 















 



 

  

   




pprkut




   

   

  

 



 















 



 

  

   




pprkut




   

   

  

 



 

  


   

   

    
 Review request changed

    
   


  

  

  






 

  Status:
  

  Closed (submitted)

  
 




 
Change Summary:

 
Pushed to master (6771878)





 


 










  

 









     

    

   

  

  

   
   Loading...